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

ci: Fix saving tag message to file in release tag workflow #110

Merged
merged 1 commit into from
Nov 2, 2021

Conversation

playpauseandstop
Copy link
Owner

Instead of saving tag_message output into tag_message.txt before calling git tag in bash shell, save that content using python shell, which results in "ignoring" backticks commands as for Python that will be just a string.

Insted of saving `tag_message` output into `tag_message.txt` before
calling `git tag` in bash shell, save that content using python shell,
which results in "ignoring" backticks commands as for Python that will
be just a string.
@playpauseandstop playpauseandstop added ci Changes to CI configuration files and scripts fix A bug fix labels Nov 2, 2021
@playpauseandstop playpauseandstop enabled auto-merge (squash) November 2, 2021 19:20
@playpauseandstop playpauseandstop merged commit cd17efe into master Nov 2, 2021
@playpauseandstop playpauseandstop deleted the ci/fix-release-tag-message branch November 2, 2021 19:24
badabump-release-bot bot pushed a commit that referenced this pull request Nov 2, 2021
Other:
------

- Fix saving tag message to file in release tag workflow (#110)
playpauseandstop added a commit that referenced this pull request Nov 2, 2021
Check that `backticks` and code snippets works well in git tags borrowed from release messages.

```py
print("Hello, world!")
```

Other:
------

- Fix saving tag message to file in release tag workflow (#110)

Co-authored-by: playpauseandstop <playpauseandstop@users.noreply.github.com>
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.109% when pulling f35b293 on ci/fix-release-tag-message into 9a65b8f on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Changes to CI configuration files and scripts fix A bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants