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

Add logic to generate Renovate postUpgradeTasks in the component template #424

Merged
merged 4 commits into from
Mar 21, 2022

Conversation

simu
Copy link
Member

@simu simu commented Mar 18, 2022

The PR contains the following changes:

  1. Remove the unused modulesync parameter githubUrl from the initial modulesync config in the component template.
  2. Add new make targets golden-diff-all and gen-golden-all in the component template
  3. Add logic to the renovate.json template in the component template to generate section postUpgradeTasks when golden tests are enabled when creating a component with commodore component new.
  4. Update the initial modulesync config generated by the component template to match the changes introduced by
    Add logic to generate Renovate postUpgradeTasks modulesync-control#62 which facilitate keeping the Renovate postUpgradeTasks updated when new matrix tests are added.

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Update tests.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • Link this PR to related issues.

The modulesync template
(https://github.com/projectsyn/modulesync-control), doesn't use the
`githubUrl` parameter anymore.

This commit removes the parameter from the initial modulesync config
generated by `commodore component new`.
@simu simu added the enhancement New feature or request label Mar 18, 2022
@simu simu changed the title Add logic to generate Renovate postUpgradeTasks in component template Add logic to generate Renovate postUpgradeTasks in the component template Mar 18, 2022
@simu simu force-pushed the feat/component-template/renovate-postupgrade-tasks branch from 7b70ceb to 80ee3e7 Compare March 18, 2022 14:05
@simu simu marked this pull request as ready for review March 18, 2022 14:15
@simu simu requested review from bastjan and glrf March 18, 2022 14:15
@simu simu force-pushed the feat/component-template/renovate-postupgrade-tasks branch from 80ee3e7 to fa1ade5 Compare March 18, 2022 14:27
simu added 3 commits March 18, 2022 16:29
…template

These targets are written to use a static list of test instances in make
variable `test_instances`. Since the initial template only has a single
test instance, we just hard code the default value of the variable in
the component template.

We use a generic recursive make target based on the `test_instances`
make variable to implement the new targets. This allows us to reuse the
existing `golden-diff` and `gen-golden` targets.

The new targets are only added for components which have matrix tests
enabled (command line flag `--matrix-tests`).
This commit adds logic to the `renovate.json` template in the component
template to generate section `postUpgradeTasks` when golden tests are
enabled when creating a component with `commodore component new`.
This commit adjusts the initial modulesync config generated by the
component template to match the changes introduced by
projectsyn/modulesync-control#62 which
facilitate keeping the Renovate `postUpgradeTasks` updated when new
matrix tests are added.
@simu simu force-pushed the feat/component-template/renovate-postupgrade-tasks branch from fa1ade5 to 1903a2d Compare March 18, 2022 15:50
@simu simu merged commit 9022ac6 into master Mar 21, 2022
@simu simu deleted the feat/component-template/renovate-postupgrade-tasks branch March 21, 2022 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants