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

clarify expected failures #1790

Merged
merged 1 commit into from
Mar 10, 2021
Merged

clarify expected failures #1790

merged 1 commit into from
Mar 10, 2021

Conversation

butonic
Copy link
Member

@butonic butonic commented Mar 10, 2021

Some features, while covered by the ownCloud 10 acceptance tests, will not be implmented for now:

see also:
owncloud/client#8398
#214
cs3org/reva#1521

Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic butonic force-pushed the clarify-expected-failures branch from e886520 to 40eec34 Compare March 10, 2021 12:16
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Contributor

@phil-davis phil-davis left a comment

Choose a reason for hiding this comment

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

If CI still passes then this must be OK!
The expected failures for EOS storage has big changes. CI does not run for EOS storage, so we don't directly know if it is exactly current.

@phil-davis phil-davis merged commit bcfd6ac into master Mar 10, 2021
@delete-merged-branch delete-merged-branch bot deleted the clarify-expected-failures branch March 10, 2021 13:15
ownclouders pushed a commit that referenced this pull request Mar 10, 2021
Merge: f4c1c06 40eec34
Author: Phil Davis <[email protected]>
Date:   Wed Mar 10 19:00:25 2021 +0545

    Merge pull request #1790 from owncloud/clarify-expected-failures

    clarify expected failures
@butonic
Copy link
Member Author

butonic commented Mar 10, 2021

yep, for EOS I tried to also file issues into File, Sync, Share, User management, Other and Won't Fix. We still need a dockerized eos setup for CI.

Related: #586 and #898

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants