Skip to content

Conversation

@christianlupus
Copy link
Collaborator

Closes #875

The main app's info XML file was removed to avoid conflicts when new releases are generated. Then, a new info file needs to be rewritten. This is handled by another workflow. When the main XML file gets modified, this might get overwritten in an not directly predictable way. Therefore the dist file was considered the single source of truth.

This PR should cover the problem by adding another test to all PRs and warn about altered/diverged XML files.

@codecov
Copy link

codecov bot commented Jan 17, 2022

Codecov Report

Merging #878 (ddd0905) into master (d2949da) will not change coverage.
The diff coverage is n/a.

❗ Current head ddd0905 differs from pull request most recent head ceffc31. Consider uploading reports for the commit ceffc31 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master     #878   +/-   ##
=======================================
  Coverage   21.22%   21.22%           
=======================================
  Files          20       20           
  Lines        1555     1555           
=======================================
  Hits          330      330           
  Misses       1225     1225           
Flag Coverage Δ
integration 7.07% <ø> (ø)
unittests 14.14% <ø> (ø)

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

@github-actions
Copy link

github-actions bot commented Jan 17, 2022

Unit Test Results

  22 files    22 suites   8m 50s ⏱️
  66 tests   66 ✔️ 0 💤 0
726 runs  726 ✔️ 0 💤 0

Results for commit ceffc31.

♻️ This comment has been updated with latest results.

@christianlupus christianlupus force-pushed the fix/875-missing-appinfo branch from d600399 to 3451945 Compare January 17, 2022 10:29
Signed-off-by: Christian Wolf <[email protected]>
@christianlupus christianlupus marked this pull request as ready for review January 17, 2022 10:36
@christianlupus christianlupus merged commit c2270dd into master Jan 17, 2022
@delete-merged-branch delete-merged-branch bot deleted the fix/875-missing-appinfo branch January 17, 2022 11:10
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.

Missing appinfo/info.xml breaks translations

2 participants