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 tool versions to output #234

Merged
merged 1 commit into from
Mar 31, 2025

Conversation

brooke-hamilton
Copy link
Contributor

This pull request includes a small change to the entrypoint.sh script to add version number outputs to the log for pyspelling and aspell to provide more information during the script execution. This is helpful to me when comparing the locally installed versions of the tools to the versions that ran as part of a GitHub workflow.

entrypoint.sh: Added commands to display the versions of pyspelling and aspell being used.

After this change, the log looks like this:

Using pyspelling on configuration outlined in >.github/config/.pyspelling.yml<
pyspelling 2.10
@(#) International Ispell Version 3.1.20 (but really Aspell 0.60.8)
Checking files matching specification outlined in: >.github/config/.pyspelling.yml<
Using aspell to spellcheck Markdown
Running Task: Markdown...

Thank you! 🤖

Signed-off-by: Brooke Hamilton <[email protected]>
@jonasbn jonasbn self-requested a review March 31, 2025 09:58
Copy link
Collaborator

@jonasbn jonasbn left a comment

Choose a reason for hiding this comment

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

Hi @brooke-hamilton

This is approved, thanks.

Changes will be included in the next coming feature release.

@jonasbn jonasbn merged commit 484959f into rojopolis:master Mar 31, 2025
1 check passed
jonasbn pushed a commit to funkill/spellcheck-github-actions that referenced this pull request Apr 1, 2025
jonasbn added a commit that referenced this pull request Apr 1, 2025
* added hunspel with en, de, es and fr dictionaries, update docs

* added russian and ukranian dicts, update docs

* updated os version in packages links

* fixed spelling

* fixed entrypoint

* Bump python from 3.13.0-slim-bookworm to 3.13.1-slim-bookworm

Bumps python from 3.13.0-slim-bookworm to 3.13.1-slim-bookworm.

---
updated-dependencies:
- dependency-name: python
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Preparing release 0.46.0, update to Docker base image

* Bump nosborn/github-action-markdown-cli from 3.3.0 to 3.4.0

Bumps [nosborn/github-action-markdown-cli](https://github.com/nosborn/github-action-markdown-cli) from 3.3.0 to 3.4.0.
- [Release notes](https://github.com/nosborn/github-action-markdown-cli/releases)
- [Commits](nosborn/github-action-markdown-cli@v3.3.0...v3.4.0)

---
updated-dependencies:
- dependency-name: nosborn/github-action-markdown-cli
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump python from 3.13.1-slim-bookworm to 3.13.2-slim-bookworm

Bumps python from 3.13.1-slim-bookworm to 3.13.2-slim-bookworm.

---
updated-dependencies:
- dependency-name: python
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Preparing release 0.47.0

* add tool versions (#234)

Signed-off-by: Brooke Hamilton <[email protected]>

* feat: add arm64 image (#218)

* feat: add arm64 image

* ci: pin actions

* ci: add sha

* ci: fix default branch name (#235)

* Added hunspell version output to entry point

* Did some testing, added more output to entrypoint, parameter specification in action.yml and some documentation

---------

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Brooke Hamilton <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: jonasbn <[email protected]>
Co-authored-by: Brooke Hamilton <[email protected]>
Co-authored-by: Shyim <[email protected]>
Co-authored-by: Jonas Brømsø <[email protected]>
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