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

Avoid treating modules as changed, when only their tests have been updated #126

Open
JamesDawson opened this issue Jul 22, 2024 · 0 comments

Comments

@JamesDawson
Copy link
Contributor

If a modules tests include references to other modules that have been changed, then the JSON files associated with those tests get updated.

The publishing logic sees those updates files and considers it as an update to the module, however, the NerdBank versioning does not and won't increment the version number which results in an error when the build tries to publish a version that already exists.

Ref: https://github.com/endjin/Endjin.RecommendedPractices.Bicep/actions/runs/10042611254/job/27753446515

Changed modules:
general/app-configuration
general/key-vault-with-secrets-access-via-groups
general/key-vault
general/storage-account
general/synapse-workspace
Processing module: general/app-configuration
  Published - OK
fatal: tag 'general/app-configuration/1.0.3' already exists
WARNING: Error tagging module 'general/app-configuration' with 'general/app-configuration/1.0.3'
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

No branches or pull requests

1 participant