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

Rename Contributor Agreement to remove space #162

Closed
wants to merge 2 commits into from
Closed

Conversation

whilp
Copy link

@whilp whilp commented Jun 17, 2018

This makes the repo somewhat easier to work with, especially for tools that cannot process files with spaces in their names.

bazelbuild/bazel#374

  • Ready for review
  • Follows CONTRIBUTING rules
  • Reviewed by Snyk internal team

What does this PR do?

Fixes #161.

Where should the reviewer start?

Just a rename.

How should this be manually tested?

Repro details in #161.

Any background context you want to provide?

See #161. ;)

What are the relevant tickets?

#161. ;)

Screenshots

n/a

Additional questions

Thank you!

This makes the repo somewhat easier to work with, especially for tools that cannot process files with spaces in their names.

bazelbuild/bazel#374
@adrukh adrukh mentioned this pull request Aug 8, 2018
3 tasks
@adrukh
Copy link
Contributor

adrukh commented Aug 8, 2018

@whilp thanks for bearing with us!

This is now resolved on the master branch, and will be published as version 1.90.2 shortly.

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