Skip to content

Lowercase the repository name.#2772

Merged
drwetter merged 1 commit intotestssl:3.2from
Tazmaniac:docker-fix
May 16, 2025
Merged

Lowercase the repository name.#2772
drwetter merged 1 commit intotestssl:3.2from
Tazmaniac:docker-fix

Conversation

@Tazmaniac
Copy link
Copy Markdown

To please buildx (docker-3.2.yml workflow)

Describe your changes

My repo path contain upercase, for which buildx complain.
buildx failed with: ERROR: invalid tag "ghcr.io/Tazmaniac/testssl.sh:3.2": repository name must be lowercase
As the github url path is case insensitive, lowercase the repo path.

What is your pull request about?

  • Bug fix
  • Improvement
  • New feature (adds functionality)
  • Breaking change (bug fix, feature or improvement that would cause existing functionality to not work as expected)
  • Typo fix
  • Documentation update
  • Update of other files

If it's a code change please check the boxes which are applicable

  • For the main program: My edits contain no tabs, indentation is five spaces and any line endings do not contain any blank chars
  • I've read CONTRIBUTING.md and Coding_Convention.md
  • I have tested this fix or improvement against >=2 hosts and I couldn't spot a problem
  • I have tested this new feature against >=2 hosts which show this feature and >=2 host which does not (in order to avoid side effects) . I couldn't spot a problem
  • For the new feature I have made corresponding changes to the documentation and / or to help()
  • If it's a bigger change: I added myself to CREDITS.md (alphabetical order) and the change to CHANGELOG.md

@drwetter drwetter merged commit bc4a1ab into testssl:3.2 May 16, 2025
2 checks passed
@Tazmaniac Tazmaniac deleted the docker-fix branch May 19, 2025 08:12
drwetter added a commit that referenced this pull request Jun 23, 2025
... which was done in #2772 . This is done for testing as building the container doesn't
work anymore, see #2810 .

Action logs say it was build but it's unclear why it is still based on 3.2rc4 .
Hence this test.

The schedule was commented out as it would not apply either here as 3.3dev is
 now the default version.
@drwetter drwetter mentioned this pull request Jun 23, 2025
13 tasks
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.

3 participants