Skip to content

Rename hidden templates#3728

Merged
deivid-rodriguez merged 1 commit intomasterfrom
rename_hidden_templates
Jun 16, 2020
Merged

Rename hidden templates#3728
deivid-rodriguez merged 1 commit intomasterfrom
rename_hidden_templates

Conversation

@deivid-rodriguez
Copy link
Copy Markdown
Contributor

Description:

Because they don't play nice with ruby-core's gitignore and cause issues with ruby-core integration. See ruby/ruby#3230.

Also, because it's consistent with other templates such as gitignore.tt.

This PR should fix CI for bundler at #3725.

Tasks:

  • Describe the problem / feature
  • Write tests
  • Write code to solve the problem
  • Get code review from coworkers / friends

I will abide by the code of conduct.

Because they don't play nice with ruby-core's gitignore and cause issues
with ruby-core integration.

Also, because it's consistent with other templates such as `gitignore.tt`.
@hsbt
Copy link
Copy Markdown
Member

hsbt commented Jun 16, 2020

👍 It's better changes for human too.

Copy link
Copy Markdown

@simi simi left a comment

Choose a reason for hiding this comment

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

Yes, those directories act as hidden for no reason in source code.

🤖 👍 + 🧑‍🤝‍🧑 👍 = 💯 👍

@deivid-rodriguez
Copy link
Copy Markdown
Contributor Author

Thanks both! ❤️

@deivid-rodriguez deivid-rodriguez merged commit 70c0b48 into master Jun 16, 2020
@deivid-rodriguez deivid-rodriguez deleted the rename_hidden_templates branch June 16, 2020 11:35
@utkarsh2102 utkarsh2102 mentioned this pull request Jun 16, 2020
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants