Skip to content

Add additional metadata to activeadmin.gemspec#8012

Merged
javierjulio merged 1 commit into
activeadmin:masterfrom
Aesthetikx:gemspec_metadata
Jul 30, 2023
Merged

Add additional metadata to activeadmin.gemspec#8012
javierjulio merged 1 commit into
activeadmin:masterfrom
Aesthetikx:gemspec_metadata

Conversation

@Aesthetikx

Copy link
Copy Markdown
Contributor

This commits adds additional metadata to activeadmin.gemspec based on the rubygems specification. This should add a few additional links to the sidebar on rubygems.org, namely Changelog, and also improve discoverability with other tools using the rubygems ecosystem.

Comment thread activeadmin.gemspec
@codecov

codecov Bot commented Jul 30, 2023

Copy link
Copy Markdown

Codecov Report

Patch and project coverage have no change.

Comparison is base (9999e80) 98.89% compared to head (4a974c5) 98.89%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #8012   +/-   ##
=======================================
  Coverage   98.89%   98.89%           
=======================================
  Files         184      184           
  Lines        4698     4698           
=======================================
  Hits         4646     4646           
  Misses         52       52           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@javierjulio javierjulio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@javierjulio javierjulio merged commit 4707eff into activeadmin:master Jul 30, 2023
@Aesthetikx Aesthetikx deleted the gemspec_metadata branch August 17, 2023 13:53
javierjulio pushed a commit that referenced this pull request Oct 9, 2023
javierjulio added a commit that referenced this pull request Oct 9, 2023
* Remove link rake task

All except for one was to do with whitespace. We want to reduce code footprint and use off-the-shelf tools for basic checks like these. Since they aren't critical we are fine without replacements other than what eslint and rubocop already offer.

The sass-rails lint was for webpacker but we have already removed webpacker support. We won't be using any images or asset URLs in the new v4 approach for assets.

* Add metadata to activeadmin.gemspec (#8012)

* Optimize data generation with insert_all

* Mention Ransack allowlist in documentation (#8043)

* Update .gitignore

* Update all outdated dependencies

Ran `bin/bundle update`

Ran `yarn upgrade-interactive --latest`

Note this updates the default gemfile to use Rails 7.1 but we are good with that for our 3-0-stable branch.

---------

Co-authored-by: John DeSilva <desilvjo@umich.edu>
Co-authored-by: Francesco Belladonna <francesco@fc5.me>
faizaankhan pushed a commit to bugcrowd/activeadmin that referenced this pull request Dec 28, 2023
* Remove link rake task

All except for one was to do with whitespace. We want to reduce code footprint and use off-the-shelf tools for basic checks like these. Since they aren't critical we are fine without replacements other than what eslint and rubocop already offer.

The sass-rails lint was for webpacker but we have already removed webpacker support. We won't be using any images or asset URLs in the new v4 approach for assets.

* Add metadata to activeadmin.gemspec (activeadmin#8012)

* Optimize data generation with insert_all

* Mention Ransack allowlist in documentation (activeadmin#8043)

* Update .gitignore

* Update all outdated dependencies

Ran `bin/bundle update`

Ran `yarn upgrade-interactive --latest`

Note this updates the default gemfile to use Rails 7.1 but we are good with that for our 3-0-stable branch.

---------

Co-authored-by: John DeSilva <desilvjo@umich.edu>
Co-authored-by: Francesco Belladonna <francesco@fc5.me>
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