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

Change edit_uri default branch to v0.x.x in mkdocs.yml #311

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

cwasicki
Copy link
Contributor

No description provided.

@cwasicki cwasicki requested a review from a team as a code owner September 27, 2024 09:10
@cwasicki cwasicki requested a review from Marenz September 27, 2024 09:10
@github-actions github-actions bot added the part:template Affects the cookiecutter template files label Sep 27, 2024
@cwasicki
Copy link
Contributor Author

Looks like I missed something, will have a look later.

llucax
llucax previously approved these changes Sep 30, 2024
Copy link
Contributor

@llucax llucax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, the tests are actually running cookiecutter and checking the output, and the TODOs are listed there, this is why is failing. Did you use the command to update the golden tests or did you update them manually? I'm not sure if that's handled by that command.

In any case, I think the TODO should stay, maybe even more now than before, as 0.x.x is not very common at all as default branch name.

@llucax llucax dismissed their stale review September 30, 2024 10:56

Missed the issue with the tests and the removed TODO

@llucax
Copy link
Contributor

llucax commented Sep 30, 2024

BTW, if this is too distracting for you, I can finish the PR myself, thanks for going straight to a PR instead of an issue!

@cwasicki
Copy link
Contributor Author

Did you use the command to update the golden tests or did you update them manually?

I did it manually.

BTW, if this is too distracting for you, I can finish the PR myself, thanks for going straight to a PR instead of an issue!

If you can quickly fix it would be highly appreciated :)

@llucax llucax self-assigned this Oct 3, 2024
@llucax llucax added this to the v0.11.0 milestone Oct 3, 2024
Signed-off-by: cwasicki <126617870+cwasicki@users.noreply.github.com>
Co-authored-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax
Copy link
Contributor

llucax commented Oct 3, 2024

Force-merging, as I only re-generated the golden tests (and applied my own suggestion 😄 )

@llucax llucax merged commit 62dc698 into frequenz-floss:v0.x.x Oct 3, 2024
14 checks passed
@llucax llucax added the type:enhancement New feature or enhancement visitble to users label Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part:template Affects the cookiecutter template files type:enhancement New feature or enhancement visitble to users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants