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

Semantic Docker Tagging #254

Merged
merged 1 commit into from
Jan 17, 2019
Merged

Semantic Docker Tagging #254

merged 1 commit into from
Jan 17, 2019

Conversation

philsturgeon
Copy link
Contributor

@philsturgeon philsturgeon commented Jan 17, 2019

Use the awesome semantic tagging approach @razvan-wework just used for another tool.

screen shot 2019-01-17 at 3 02 50 pm

@philsturgeon philsturgeon force-pushed the new-docker-versions branch 4 times, most recently from 2ee13d0 to 80c6ba0 Compare January 17, 2019 19:55
@philsturgeon philsturgeon changed the title Only report to coveralls if not a fork Semantic Tagging Jan 17, 2019
@philsturgeon philsturgeon changed the title Semantic Tagging Semantic Docker Tagging Jan 17, 2019
@philsturgeon philsturgeon merged commit ba42c55 into master Jan 17, 2019
@philsturgeon philsturgeon deleted the new-docker-versions branch January 17, 2019 20:11
@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.626% when pulling 80c6ba0 on new-docker-versions into 2971633 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.626% when pulling 80c6ba0 on new-docker-versions into 2971633 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.626% when pulling e6cda75 on new-docker-versions into 2971633 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.626% when pulling 80c6ba0 on new-docker-versions into 2971633 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.626% when pulling 80c6ba0 on new-docker-versions into 2971633 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.626% when pulling e6cda75 on new-docker-versions into 2971633 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.626% when pulling e6cda75 on new-docker-versions into 2971633 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.626% when pulling e6cda75 on new-docker-versions into 2971633 on master.

- run:
name:
command: |
docker_tag=$DOCKER_REPOSITORY:${CIRCLE_SHA1:0:7}

Choose a reason for hiding this comment

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

this ${CIRCLE_SHA1:0:7} has been identified as not working for 100% of shells, use $CIRCLE_SHA1 | cut -c1-7 instead

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.

4 participants