Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Yaml changes stabikat 2024 08 #106

Merged
merged 4 commits into from
Sep 13, 2024
Merged

Yaml changes stabikat 2024 08 #106

merged 4 commits into from
Sep 13, 2024

Conversation

annekauf
Copy link
Collaborator

Productive Updates added to our current status of yaml on main.

@annekauf
Copy link
Collaborator Author

annekauf commented Sep 11, 2024 •
edited
Loading

grafik

@annekauf
Partial Reverse Update Highlighting
71e6b00 
commented out lines 469-471
@annekauf
Copy link
Collaborator Author

annekauf commented Sep 11, 2024 •
edited
Loading

Removal of productive update highlighting: 4 more passed tests

grafik

@annekauf
Reverse Productive Update BKL Browse
13de5ed 
in addition to previous commit removal of all BKL Browse addition from productive update
#58?
@annekauf
Copy link
Collaborator Author

annekauf commented Sep 11, 2024 •
edited
Loading

Removing productive updates BKL browse in yaml - Commit has no impact on #58 but gets one additional passed test (in topical.cy.js)
BUT: is part of productive Stabikat, so probably can't be removed. Delete this Commit?
grafik

13de5ed

@duncdrum duncdrum mentioned this pull request Sep 13, 2024
Open
@duncdrum duncdrum self-requested a review September 13, 2024 13:09
duncdrum
duncdrum approved these changes Sep 13, 2024
View reviewed changes
Copy link
Member

@duncdrum duncdrum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see #109

let's continue like this

@duncdrum duncdrum merged commit 3ac056c into main Sep 13, 2024
2 checks passed
@duncdrum duncdrum deleted the yaml-changes-stabikat-2024_08 branch September 13, 2024 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@duncdrum duncdrum duncdrum approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@annekauf @duncdrum