Skip to content

Update specification reference required fields and metadata docs#187

Closed
greysteil wants to merge 1 commit into
rubygems:gh-pagesfrom
greysteil:update-specification-reference
Closed

Update specification reference required fields and metadata docs#187
greysteil wants to merge 1 commit into
rubygems:gh-pagesfrom
greysteil:update-specification-reference

Conversation

@greysteil
Copy link
Copy Markdown

@greysteil greysteil commented Jun 22, 2017

Generated from the changes in ruby/rubygems#1960.

Note: the metadata docs were out-of-date, as changes had been made to Gem::Specification previously and not pulled in here.

@sonalkr132
Copy link
Copy Markdown
Member

@greysteil did you use the rubygems master for these changes? I think this repo gets updated only after changes are released. I have been waiting for months 😭 (Unreleased commits on rubygems)

@greysteil
Copy link
Copy Markdown
Author

greysteil commented Jun 22, 2017

Hmm, I did just use master. Can someone confirm that's the process, and if so whether PRs to update the Specification Reference docs are needed (rather than them being auto-generated)?

bundlerbot added a commit to ruby/rubygems that referenced this pull request Aug 31, 2017
Update specification reference docs

# Description:

Previously, the Specification Reference [guide](http://guides.rubygems.org/specification-reference/#required-gemspec-attributes) listed incorrect required attributes (as noted in rubygems/guides#121, for example).

This PR reorders `lib/rubygems/specification.rb` in order to create a better specification-reference.
- Updates the required attributes to be just `name`, `version`, `summary` and `files`, in line with the `@@required_attributes` and `@@default_value` variables.
- Cleans up doc consistency by creating a `Recommended gemspec attributes` section, rather than relying on `:category: Recommended gemspec attributes` tags.
- Adds a `metadata` hash to the headline example gemspec.

Corresponding guides PR is rubygems/guides#187.

______________

# Tasks:

- [x] Describe the problem / feature
- [x] Write code to solve the problem
- [x] 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).
@duckinator
Copy link
Copy Markdown
Contributor

hi. I'll ask around and see if I can figure out what the process is for publishing updates to guides. then maybe I can help resolve all the PRs on this repo.

@duckinator
Copy link
Copy Markdown
Contributor

looks like they get pulled in when there's a RubyGems release.

@duckinator
Copy link
Copy Markdown
Contributor

closing since this should be handled with the next RubyGems release.

thanks for the help, @greysteil. ❤️

@duckinator duckinator closed this Sep 1, 2017
@greysteil greysteil deleted the update-specification-reference branch September 1, 2017 15:37
hsbt pushed a commit to ruby/rubygems-server that referenced this pull request Sep 24, 2021
Update specification reference docs

# Description:

Previously, the Specification Reference [guide](http://guides.rubygems.org/specification-reference/#required-gemspec-attributes) listed incorrect required attributes (as noted in rubygems/guides#121, for example).

This PR reorders `lib/rubygems/specification.rb` in order to create a better specification-reference.
- Updates the required attributes to be just `name`, `version`, `summary` and `files`, in line with the `@@required_attributes` and `@@default_value` variables.
- Cleans up doc consistency by creating a `Recommended gemspec attributes` section, rather than relying on `:category: Recommended gemspec attributes` tags.
- Adds a `metadata` hash to the headline example gemspec.

Corresponding guides PR is rubygems/guides#187.

______________

# Tasks:

- [x] Describe the problem / feature
- [x] Write code to solve the problem
- [x] 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.

3 participants