Skip to content

[TAR-571] Remove the ads#106

Merged
seemethere merged 2 commits intodocker:masterfrom
seemethere:remove_ads
Mar 15, 2019
Merged

[TAR-571] Remove the ads#106
seemethere merged 2 commits intodocker:masterfrom
seemethere:remove_ads

Conversation

@seemethere
Copy link
Copy Markdown
Contributor

This reverts commit 40b1b76, reversing
changes made to 26dda3d.

@seemethere seemethere requested a review from a team March 15, 2019 17:26
@GordonTheTurtle
Copy link
Copy Markdown

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "remove_ads" git@github.com:seemethere/docker-install.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@seemethere seemethere changed the title Remove the ads [TAR-571] Remove the ads Mar 15, 2019
This reverts commit 40b1b76, reversing
changes made to 26dda3d.

Signed-off-by: Eli Uriegas <eli.uriegas@docker.com>
Comment thread Jenkinsfile Outdated
'aarch64-verify-install-ubuntu-xenial',
'aarch64-verify-install-ubuntu-bionic',
'aarch64-verify-install-centos-7',
'aarch64-verify-install-fedora-29',
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we reintroduce the fedora 29?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No unfortunately I think that was the side-effect of adding this on top of PR #95, going to re-apply that particular patch

Signed-off-by: Eli Uriegas <eli.uriegas@docker.com>
(cherry picked from commit eabd8aa)
Signed-off-by: Eli Uriegas <eli.uriegas@docker.com>
Copy link
Copy Markdown
Member

@StefanScherer StefanScherer left a comment

Choose a reason for hiding this comment

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

LGTM

@seemethere seemethere merged commit 33e2f7d into docker:master Mar 15, 2019
@seemethere seemethere deleted the remove_ads branch August 28, 2020 19:02
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.

5 participants