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

Remove .dockerignore #10642

Merged
merged 1 commit into from
Jun 2, 2021
Merged

Remove .dockerignore #10642

merged 1 commit into from
Jun 2, 2021

Conversation

miry
Copy link
Contributor

@miry miry commented Apr 17, 2021

There was a change to move Dockerfiles to separate repositories in #6141.
It does not make sense to keep the .dockerignore.
The last change was 6 years ago.

There was a change to move Dockerfiles to separate repositories in #6141.
It does not make sense to keep the `.dockerignore`. 
The last change was 6 years ago.
@straight-shoota straight-shoota added kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:infrastructure labels Apr 17, 2021
@miry
Copy link
Contributor Author

miry commented Jun 1, 2021

I could not merge.

The base branch restricts merging to authorized users. Learn more about protected branches.

@straight-shoota straight-shoota added this to the 1.1.0 milestone Jun 1, 2021
@straight-shoota
Copy link
Member

Yes, merging access is restricted to Core Team members and we follow the approval process outlined in the contribution guide.

@straight-shoota straight-shoota merged commit b22b00f into crystal-lang:master Jun 2, 2021
@miry miry deleted the patch-1 branch June 2, 2021 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants