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

Release/0.31.2 #554

Merged
merged 38 commits into from
Oct 16, 2024
Merged

Release/0.31.2 #554

merged 38 commits into from
Oct 16, 2024

Conversation

n-dusan
Copy link
Contributor

@n-dusan n-dusan commented Oct 16, 2024

Description (e.g. "Related to ...", etc.)

Code review checklist (for code reviewer to complete)

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • Commit messages are meaningful (see this for details)
  • Tests have been included and/or updated, as appropriate
  • Docstrings have been included and/or updated, as appropriate
  • Changelog has been updated, as needed (see CHANGELOG.md)

renatav and others added 30 commits October 4, 2024 18:58
…f the auth repo due to an issue with our TUF fork
We will not be releasing taf scripts CLI this release.

This reverts commit 96c0436.
Refactor config.url -> config.remote_url. This flag is set when running taf repo clone command from CLI.

The current system with taf repo clone uses the config.remote_url to clone the top level authentication repository.
However, for other repositories, we need to try cloning from all the urls in mirrors.json, not just the url that's passed in by the CLI.
To resolve, add config.clone_urls attribute that's set when repos are instantiated using repositoriesdb.
renatav and others added 8 commits October 16, 2024 16:40
It was previously retrieved through auth_repo.parent.parent, which works in most cases. The only case where this fails is when the authentication repository is not within 2 levels above the current directory, and is the current case on our AppVeyor CI.
@n-dusan n-dusan requested a review from renatav October 16, 2024 22:47
@n-dusan n-dusan self-assigned this Oct 16, 2024
@renatav renatav merged commit 0fba858 into master Oct 16, 2024
6 checks passed
@renatav renatav deleted the release/0.31.2 branch October 16, 2024 22:47
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.

2 participants