Skip to content

Lowercase for Docker build Name in documentation #567

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

Merged
merged 1 commit into from
Feb 7, 2021

Conversation

Pierre48
Copy link
Contributor

@Pierre48 Pierre48 commented Feb 4, 2021

Name components may contain lowercase letters, digits and separators.
https://docs.docker.com/engine/reference/commandline/tag/

@CLAassistant
Copy link

CLAassistant commented Feb 4, 2021

CLA assistant check
All committers have signed the CLA.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 4, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@nicolas63
Copy link
Member

thanks @Pierre48 and congratulations for your first contribution here ! 🎉
Can you sign the CLA and i merge your PR

@Pierre48
Copy link
Contributor Author

Pierre48 commented Feb 7, 2021

Done :)

@nicolas63 nicolas63 merged commit b9a98ba into jhipster:master Feb 7, 2021
@nicolas63
Copy link
Member

@Pierre48 Sorry I didn't check the target branch. You have target master but the new github convention is main can you reopen your PR on the main ? We have kept the master for the moment but it will be deleted soon

@nicolas63 nicolas63 mentioned this pull request Feb 7, 2021
4 tasks
nicolas63 added a commit that referenced this pull request Feb 7, 2021
A-5ingh pushed a commit to A-5ingh/jhipster-dotnetcore that referenced this pull request Feb 19, 2025
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