Skip to content

[Task #876] Add license scanning setup to the template #23

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

anamarijabalaban
Copy link

@anamarijabalaban anamarijabalaban commented Jun 18, 2025

Summary

Introduce a license scanning to the Rails template.

Related issue: None

Changes

Type

  • Feature: This pull request introduces a new feature.
  • Bug fix: This pull request fixes a bug.
  • Refactor: This pull request refactors existing code.
  • Documentation: This pull request updates documentation.
  • Other: This pull request makes other changes.

Additional information

  • This pull request introduces a breaking change.

Description

Add license_finder recipe and run license_finder commands in run_generators.rb.

Checklist

  • I have performed a self-review of my own code.
  • I have tested my changes, including edge cases.
  • I have added necessary tests for the changes introduced (if applicable).
  • I have updated the documentation to reflect my changes (if applicable).

Additional notes

@anamarijabalaban anamarijabalaban requested a review from a team as a code owner June 18, 2025 09:20
@anamarijabalaban anamarijabalaban requested a review from vr4b4c June 18, 2025 09:21
@anamarijabalaban anamarijabalaban self-assigned this Jun 18, 2025
@anamarijabalaban anamarijabalaban force-pushed the feature/876-add-license-scanning branch from 2e21857 to beff1fc Compare June 18, 2025 09:26
Comment on lines +12 to +19
LicenseFinder:
enabled: true
on_warn: fail
command: ['bundle', 'exec', 'license_finder']
requires_files: true
include:
- 'Gemfile*'
- 'package.json'
Copy link
Member

Choose a reason for hiding this comment

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

I think this needs to be indented one more level to the left to be correctly nested under PreCommit section.


append_to_file 'Gemfile', after: /gem 'rubocop-infinum'.*\n/ do
<<~HEREDOC
gem 'license_finder', require: false
Copy link
Member

Choose a reason for hiding this comment

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

Indentation problem here as well

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