Skip to content

Conversation

@skjnldsv
Copy link
Member

4.2.0

Changes

Fixes

Other

  • Various dependencies

Full Changelog: v4.1.0...v4.2.0

Signed-off-by: skjnldsv <[email protected]>
@skjnldsv skjnldsv self-assigned this Feb 28, 2025
@skjnldsv skjnldsv added the 3. to review Waiting for reviews label Feb 28, 2025
@skjnldsv skjnldsv enabled auto-merge February 28, 2025 08:05

### Changes
* feat: migrate owncloud guests-users to nextcloud guests by @skjnldsv in https://github.com/nextcloud/guests/pull/1286
* feat(deps): Add Nextcloud 32 support by @nickvergessen in https://github.com/nextcloud/guests/pull/1282
Copy link
Member

Choose a reason for hiding this comment

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

Hmm not sure you want to release this already?

Copy link
Member Author

Choose a reason for hiding this comment

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

We usually do no ?
if in alpha/beta, it won't matter anyway

Copy link
Member Author

Choose a reason for hiding this comment

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

When we'll get closer to 32, we'll have to release a new one anyway which will double check if still compatible

Copy link
Member Author

Choose a reason for hiding this comment

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

What else are we supposed to do, shall we revert #1282 ?

Copy link
Member

Choose a reason for hiding this comment

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

yeah in some apps I stopped merging them 🙈
For multi branch it's really annoying, either during development or releasing.

Copy link
Member

Choose a reason for hiding this comment

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

Maybe a fix would be in server to allow enabling without --force when you run from git?

Copy link
Member Author

Choose a reason for hiding this comment

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

I mean, isn't force what we expect to have for our daily anyway?
That's exactly the reason we have that option, no?

I actually don't get Tobias's comment on #1282 🤔

@skjnldsv skjnldsv merged commit 8e7468b into master Feb 28, 2025
45 checks passed
@skjnldsv skjnldsv deleted the feat/release branch February 28, 2025 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants