Skip to content

Remove old workaround for non-git environments#3980

Merged
deivid-rodriguez merged 1 commit intomasterfrom
no_bundler_gemspec_rewrite
Oct 2, 2020
Merged

Remove old workaround for non-git environments#3980
deivid-rodriguez merged 1 commit intomasterfrom
no_bundler_gemspec_rewrite

Conversation

@deivid-rodriguez
Copy link
Copy Markdown
Contributor

Description:

I noticed that running ruby setup.rb was rewriting the bundler.gemspec when I run it on Windows on a shared folder and noticed that all line feeds had been rewritten.

This is no longer necessary since we no longer use git in bundler's gemspec.

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.

@bundlerbot bundlerbot added CI CI related issues RubyGems labels Sep 30, 2020
@deivid-rodriguez deivid-rodriguez force-pushed the no_bundler_gemspec_rewrite branch from b1f383a to ed5483d Compare October 2, 2020 09:13
I noticed that running `ruby setup.rb` was rewriting the
`bundler.gemspec` when I run it on Windows on a shared folder and
noticed that all line feeds had been rewritten.

This is no longer necessary since we no longer use `git` in bundler's
gemspec.
@deivid-rodriguez deivid-rodriguez force-pushed the no_bundler_gemspec_rewrite branch from ed5483d to f811599 Compare October 2, 2020 10:49
@deivid-rodriguez deivid-rodriguez merged commit da6a681 into master Oct 2, 2020
@deivid-rodriguez deivid-rodriguez deleted the no_bundler_gemspec_rewrite branch October 2, 2020 12:11
deivid-rodriguez added a commit that referenced this pull request Dec 7, 2020
Remove old workaround for non-git environments

(cherry picked from commit da6a681)
deivid-rodriguez added a commit that referenced this pull request Dec 7, 2020
Remove old workaround for non-git environments

(cherry picked from commit da6a681)
deivid-rodriguez added a commit that referenced this pull request Dec 7, 2020
Remove old workaround for non-git environments

(cherry picked from commit da6a681)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants