Skip to content

Docker Cloud: update tag mapping label, some typo fixes #354

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
Oct 26, 2016

Conversation

sanscontext
Copy link
Contributor

Updates the Tag Mapping label to the new "Build Rules". Plus some copy edits and typo fixes.

@pkennedyr @pchico83

Signed-off-by: LRubin [email protected]

If you configured a regex to select the source, you can reference the
capture groups and use its result as part of the tag. To learn more, see
[regexes](automated-build.md#regexes-and-automated-builds).

* Specify the **Dockerfile location** as a path relative to the root of the source code repository. (If the Dockerfile is at the repository root, leave this path set to `/`.)

Choose a reason for hiding this comment

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

it is relative to the build context, not to the root

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Context isn't live yet. That update will happen later.

Choose a reason for hiding this comment

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

ok, as long as it is changed before going live.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That'll be in a separate PR, as discussed. :)

@sanscontext sanscontext merged commit eb84a42 into docker:master Oct 26, 2016
@sanscontext sanscontext deleted the tag-mapping branch October 26, 2016 21:04
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