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

Cleanup deprecated style #2193

Merged
merged 12 commits into from Feb 16, 2018
Merged

Cleanup deprecated style #2193

merged 12 commits into from Feb 16, 2018

Conversation

hsbt
Copy link
Member

@hsbt hsbt commented Feb 16, 2018

Description:

Followed up to remove old ruby versions. I did clean up redundant/deprecated sections on gemspec and travis.yml.


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.

@hsbt
Copy link
Member Author

hsbt commented Feb 16, 2018

@bundlerbot r+

@bundlerbot
Copy link
Collaborator

📌 Commit 3dc5728 has been approved by hsbt

@bundlerbot
Copy link
Collaborator

⌛ Testing commit 3dc5728 with merge 263a902...

bundlerbot added a commit that referenced this pull request Feb 16, 2018
Cleanup deprecated style

# Description:

Followed up to remove old ruby versions. I did clean up redundant/deprecated sections on gemspec and travis.yml.
______________

# 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](https://github.com/rubygems/rubygems/blob/master/CODE_OF_CONDUCT.md).
@hsbt
Copy link
Member Author

hsbt commented Feb 16, 2018

@bundlerbot r+

@bundlerbot
Copy link
Collaborator

💡 This pull request was already approved, no need to approve it again.

  • This pull request is currently being tested. If there's no response from the continuous integration service, you may use retry to trigger a build again.

@bundlerbot
Copy link
Collaborator

📌 Commit 3dc5728 has been approved by hsbt

@hsbt
Copy link
Member Author

hsbt commented Feb 16, 2018

@bundlerbot retry

@bundlerbot
Copy link
Collaborator

⌛ Testing commit 3dc5728 with merge a6415f7...

bundlerbot added a commit that referenced this pull request Feb 16, 2018
Cleanup deprecated style

# Description:

Followed up to remove old ruby versions. I did clean up redundant/deprecated sections on gemspec and travis.yml.
______________

# 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](https://github.com/rubygems/rubygems/blob/master/CODE_OF_CONDUCT.md).
@bundlerbot
Copy link
Collaborator

☀️ Test successful - status-travis
Approved by: hsbt
Pushing a6415f7 to master...

@bundlerbot bundlerbot merged commit 3dc5728 into master Feb 16, 2018
@segiddins segiddins deleted the cleanup-deprecated-style branch February 16, 2018 06:33
@@ -1,44 +1,36 @@
# -*- encoding: utf-8 -*-
# stub: rubygems-update 2.7.4 ruby hide_lib_for_update

Gem::Specification.new do |s|
s.name = "rubygems-update".freeze
s.version = "2.7.4"
Copy link
Member

Choose a reason for hiding this comment

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

should this be reading from lib/rubygems.rb?

s.rdoc_options = ["--main".freeze, "README.md".freeze, "--title=RubyGems Update Documentation".freeze]
s.extra_rdoc_files = [
Copy link
Member

Choose a reason for hiding this comment

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

should this include the bundler manpages?

"bundler/CONTRIBUTING.md".freeze, "bundler/LICENSE.md".freeze, "bundler/README.md".freeze,
"hide_lib_for_update/note.txt".freeze,
]

s.required_ruby_version = Gem::Requirement.new(">= 2.2.2".freeze)
s.rubygems_version = "2.7.3".freeze
Copy link
Member

Choose a reason for hiding this comment

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

I believe this should be removed, it's set by RubyGems when building

hsbt added a commit that referenced this pull request Mar 3, 2018
@hsbt hsbt mentioned this pull request Mar 3, 2018
4 tasks
bundlerbot added a commit that referenced this pull request Mar 5, 2018
Improve gemspec and package task.

# Description:

Followed up #2193 (review) and Improve package task.
______________

# Tasks:

- [x] Describe the problem / feature
- [x] Write tests
- [ ] Write code to solve the problem
- [ ] Get code review from coworkers / friends

I will abide by the [code of conduct](https://github.com/rubygems/rubygems/blob/master/CODE_OF_CONDUCT.md).
hsbt pushed a commit to rubygems/rubygems-server that referenced this pull request Sep 24, 2021
Improve gemspec and package task.

# Description:

Followed up rubygems/rubygems#2193 (review) and Improve package task.
______________

# Tasks:

- [x] Describe the problem / feature
- [x] Write tests
- [ ] Write code to solve the problem
- [ ] Get code review from coworkers / friends

I will abide by the [code of conduct](https://github.com/rubygems/rubygems/blob/master/CODE_OF_CONDUCT.md).
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.

None yet

4 participants