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

projects: Add support for custom data #4282

Conversation

sergio-costas
Copy link
Contributor

@sergio-costas sergio-costas commented Jul 17, 2023

This patch implements proposal ST090. It allows to add custom metadata into the snapcraft.yaml files without interferring with snapcraft. It works both in legacy and modern files.

This is useful for adding extra data for tools that manage snap files, like updatesnap.py, which searches for new versions of the parts in a snapcraft.yaml file.

Proposal document: https://docs.google.com/document/d/1QX0moP6V709D7L267pIT5AREQetbPb7KsFrpgvCLKVM

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run make lint?
  • Have you successfully run pytest tests/unit?

@sergio-costas sergio-costas force-pushed the DT-1273-add-support-for-custom-extensions-in-snapcrafts-yaml-files branch 3 times, most recently from d1321b6 to 48ed9a7 Compare July 18, 2023 11:33
@codecov-commenter
Copy link

codecov-commenter commented Jul 18, 2023

Codecov Report

Merging #4282 (0fa2d76) into main (dbe76f2) will increase coverage by 0.00%.
The diff coverage is 95.23%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main    #4282   +/-   ##
=======================================
  Coverage   88.99%   88.99%           
=======================================
  Files         296      296           
  Lines       20208    20227   +19     
=======================================
+ Hits        17984    18001   +17     
- Misses       2224     2226    +2     
Impacted Files Coverage Δ
snapcraft/utils.py 91.79% <94.44%> (+0.22%) ⬆️
snapcraft/projects.py 96.12% <100.00%> (+<0.01%) ⬆️
snapcraft_legacy/project/_project_info.py 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sergio-costas sergio-costas marked this pull request as ready for review July 18, 2023 15:27
@sergio-costas sergio-costas changed the title Add support for custom data projects: Add support for custom data Jul 24, 2023
This patch implements proposal ST090. It allows to add custom
metadata into the snapcraft.yaml files without interferring
with snapcraft.

This is useful for adding extra data for tools that manage
snap files, like updatesnap.py, which searches for new versions
of the parts in a snapcraft.yaml file.

Proposal: https://docs.google.com/document/d/1QX0moP6V709D7L267pIT5AREQetbPb7KsFrpgvCLKVM
@sergio-costas sergio-costas force-pushed the DT-1273-add-support-for-custom-extensions-in-snapcrafts-yaml-files branch from 075e8c8 to 0fa2d76 Compare July 24, 2023 09:07
@sergiusens
Copy link
Collaborator

Out of curiosity, why are you not using renovate to handle these things?

@sergio-costas
Copy link
Contributor Author

@sergiusens Ehm... well, I presume that this is the moment where I discover that there is already a tool for what I needed... What is "renovate"?

@sergiusens
Copy link
Collaborator

https://github.com/renovatebot/renovate

Closing this for now, as this seems out of scope

@sergiusens sergiusens closed this Aug 11, 2023
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.

3 participants