X Tutup
Skip to content

Very minimal empty ODF files#37896

Merged
micbar merged 4 commits intoowncloud:masterfrom
timar:master
Sep 14, 2020
Merged

Very minimal empty ODF files#37896
micbar merged 4 commits intoowncloud:masterfrom
timar:master

Conversation

@timar
Copy link
Contributor

@timar timar commented Sep 8, 2020

These files contain no data at all. This guarantees that when the
user opens a new document in the richdocuments app, the language
of paragraph, page size, cell date format, currency, etc. will be
according to the current locale.

These files contain no data at all. This guarantees that when the
user opens a new document in the richdocuments app, the language
of paragraph, page size, cell date format, currency, etc. will be
according to the current locale.
@CLAassistant
Copy link

CLAassistant commented Sep 8, 2020

CLA assistant check
All committers have signed the CLA.

@mmattel
Copy link
Contributor

mmattel commented Sep 10, 2020

@phil-davis can you retrigger codecov, seems to be stuck...

@phil-davis
Copy link
Contributor

codecov/patch is always stuck these days - it always says "In progress - This step has started..." and no result comes. I have had to override the result for every PR to core in the last week or more.

@phil-davis
Copy link
Contributor

@micbar please get the appropriate person to review this...

@micbar
Copy link
Contributor

micbar commented Sep 10, 2020

@phil-davis @DeepDiver1975 Background? Changelog?

@mmattel
Copy link
Contributor

mmattel commented Sep 10, 2020

@timar can you please file a changelog, see: https://github.com/owncloud/core/blob/master/changelog/README.md

@phil-davis
Copy link
Contributor

@phil-davis @DeepDiver1975 Background? Changelog?

I don't know - I just got mentioned regarding codecov.

Signed-off-by: Andras Timar <andras.timar@collabora.com>
@codecov
Copy link

codecov bot commented Sep 10, 2020

Codecov Report

Merging #37896 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #37896   +/-   ##
=========================================
  Coverage     64.75%   64.75%           
  Complexity    19403    19403           
=========================================
  Files          1285     1285           
  Lines         75823    75823           
  Branches       1336     1336           
=========================================
  Hits          49096    49096           
  Misses        26333    26333           
  Partials        394      394           
Flag Coverage Δ Complexity Δ
#javascript 54.06% <ø> (ø) 0.00 <ø> (ø)
#phpunit 65.93% <ø> (ø) 19403.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd8d461...1ba49e3. Read the comment docs.

timar and others added 2 commits September 10, 2020 13:08
Co-authored-by: Phil Davis <phil@jankaritech.com>
@owncloud owncloud deleted a comment from update-docs bot Sep 10, 2020
Copy link
Contributor

@mmattel mmattel left a comment

Choose a reason for hiding this comment

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

LGTM (but not tested with the richdocument app because not installed)

@codecov
Copy link

codecov bot commented Sep 10, 2020

Codecov Report

Merging #37896 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #37896   +/-   ##
=========================================
  Coverage     64.75%   64.75%           
  Complexity    19403    19403           
=========================================
  Files          1285     1285           
  Lines         75823    75823           
  Branches       1336     1336           
=========================================
  Hits          49096    49096           
  Misses        26333    26333           
  Partials        394      394           
Flag Coverage Δ Complexity Δ
#javascript 54.06% <ø> (ø) 0.00 <ø> (ø)
#phpunit 65.93% <ø> (ø) 19403.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd8d461...9e45084. Read the comment docs.

@phil-davis
Copy link
Contributor

Stupid codecov. I tried to fake the results, but it comes in "over the top" with its own silly result.

@micbar please force merge.

@mmattel
Copy link
Contributor

mmattel commented Sep 14, 2020

@micbar ping (force merge, see comment of @phil-davis above)

@micbar micbar merged commit 9068141 into owncloud:master Sep 14, 2020
JammingBen pushed a commit that referenced this pull request Nov 23, 2020
Signed-off-by: Andras Timar <andras.timar@collabora.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

X Tutup