-
Notifications
You must be signed in to change notification settings - Fork 187
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.
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
chore: bump web to v7.1.1 #7601
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to make sure to not delete the release-7.1.1
branch in web. So if you want to merge the version bump back to the stable-7.1
branch, you could either branch off from the release-7.1.1
branch or cherry-pick the commit. :-) If you want me to set up branch protection for the release-7.1.1
branch instead, please let me know
Thanks for spotting this @kulmann Can we set the CI to |
This is only used for CI anyway, right? Then I agree, we can continue to use @kulmann Hmm I'd prefer to cherry-pick the commit(s) to |
3a7c47f
to
7f751f1
Compare
@ScharfViktor Do you have an idea why https://drone.owncloud.com/owncloud/ocis/28187/38/12 is failing? It passes locally and in CI on Web branch |
@JammingBen did you port this PR to stable-branch? owncloud/web#9587 test waits selected file like on screen: |
owncloud/web#9587 is on |
7f751f1
to
9cc621a
Compare
Apparently that's not the case 🙈 So I reverted the target branch to |
there's something wrong with that because file openes direct on text editor. test expects another behavior How reproduce:
|
Kudos, SonarCloud Quality Gate passed! |
It seems to work now, the target |
I also enabled protection for the |
chore: bump web to v7.1.1
Bumps Web to
v7.1.1
. This includes the following fixes: