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.

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

15.0.0 beta 1 #12310

Merged
merged 1 commit into from
Nov 8, 2018
Merged

15.0.0 beta 1 #12310

merged 1 commit into from
Nov 8, 2018

Conversation

MorrisJobke
Copy link
Member

@MorrisJobke MorrisJobke commented Nov 6, 2018

Critical changed for devs & admins: #11045

Pending issues:

Pending enhancements:

Pending technical debt PRs:

Pending bug fix PRs:

cc @danxuliu @rullzer @blizzz @icewind1991 @schiessle @nickvergessen @jancborchardt @skjnldsv @danielkesselberg @weeman1337 @karlitschek @jospoortvliet @tobiasKaminsky @camilasan @mario @ChristophWurst

@MorrisJobke MorrisJobke added the 2. developing Work in progress label Nov 6, 2018
@MorrisJobke MorrisJobke added this to the Nextcloud 15 milestone Nov 6, 2018
@tobiasKaminsky
Copy link
Member

For mobile we would need
#8477
#11138

We discussed this at our NC15 meeting, but even though I did some reminders, nothing was done. So I doubt that this will get 😞

@MorrisJobke
Copy link
Member Author

For mobile we would need
#8477
#11138

We discussed this at our NC15 meeting, but even though I did some reminders, nothing was done. So I doubt that this will get 😞

Yes - does not good to be solved within 8 hours :/

@georgehrke
Copy link
Member

Removed:

Because first one will have to wait for 16 and the second is a bug fix anyway, so we can always backport it.

@nickvergessen
Copy link
Member

I removed "#11871 Collaboration resources" as discussed with frank. Will be delayed to 16

@MorrisJobke
Copy link
Member Author

I separated the list above to enhancements, tech debt and bug fixes. So we can focus today and tomorrow morning on the enhancements.

@tobiasKaminsky
Copy link
Member

Looking at pending enhancements half of it is related to apps, that are currently shipped with release.
Wouldn't it make sense to release the apps independent? Of course we can bundle them into release, but having it separately means that we only need any NC15 capable app version.
Same like other apps, like News, Mail, …

@MorrisJobke
Copy link
Member Author

Moved all of them to 16.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke MorrisJobke added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Nov 8, 2018
@MorrisJobke
Copy link
Member Author

Rebased - ready for review.

Let's do the beta 1 today and accept that it could be a bit more rough. Everything that is an enhancement will go into 16. For fixes we should evaluate if they are fine to go into master.

@MorrisJobke
Copy link
Member Author

MorrisJobke commented Nov 8, 2018

Those are the failing tests of the first run:

  • tests/acceptance/features/users.feature:24
  • tests/acceptance/features/app-files-tags.feature:4
  • tests/acceptance/features/app-files-tags.feature:11
  • tests/acceptance/features/app-files-tags.feature:31
  • tests/acceptance/features/app-files-tags.feature:59
  • timeout in app-files

Let's trigger a rerun and see how the second run goes and check if there are overlapping issues.

https://drone.nextcloud.com/nextcloud/server/12535

@danxuliu
Copy link
Member

danxuliu commented Nov 8, 2018

Let's trigger a rerun and see how the second run goes and check if there are overlapping issues.

  • acceptance-app-files: timeout; there were no failures in the previous run, so it should be safe. Whoops, I missed that you wrote that there was a timeout in the previous run; I will run them in local to check. - Update: the tests fail, but because they were not adjusted to the changes in Better primary visual on public pages #12316; I will send a pull request with the fix (although this should not block the Beta 1, as the problem is only in the test themselves).
  • acceptance-app-files-tags: known problems, will be fixed but not for Beta 1.
  • acceptance-users: our friend sh: 1: kill: No such process, ignore.
  • sqlite-php7.0-samba-native: codecov timeout, ignore.

@MorrisJobke
Copy link
Member Author

Thanks - this is good to be merged now 👍 Let's build it and check out how it works

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.

7 participants