diff options
author | Josh Cooper <737664+joshcooper@users.noreply.github.com> | 2023-12-01 12:02:37 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-12-01 12:02:37 -0800 |
commit | b37c6fa2dc9396c7fdbfa8dc492193c3294f915d (patch) | |
tree | 97345a71c01592d27733a03f967e1ac2f49b94da | |
parent | 907d28ca583e9d38318f7a79f9d1904b979e13f3 (diff) | |
parent | 3e7ea029311fb5b7682761748efe06d771a5dbbf (diff) | |
download | puppet-hosts_core-b37c6fa2dc9396c7fdbfa8dc492193c3294f915d.tar.gz puppet-hosts_core-b37c6fa2dc9396c7fdbfa8dc492193c3294f915d.tar.bz2 |
Merge pull request #43 from mhashizume/PA-5395/main/github_issues
(PA-5395) Issues housekeeping
-rw-r--r-- | .github/workflows/jira.yml | 19 | ||||
-rw-r--r-- | .github/workflows/static_code_analysis.yaml | 40 | ||||
-rw-r--r-- | metadata.json | 2 | ||||
-rw-r--r-- | rakelib/commits.rake | 25 |
4 files changed, 22 insertions, 64 deletions
diff --git a/.github/workflows/jira.yml b/.github/workflows/jira.yml new file mode 100644 index 0000000..a0d49d6 --- /dev/null +++ b/.github/workflows/jira.yml @@ -0,0 +1,19 @@ +--- +name: Export issue to Jira + +on: + issues: + types: [labeled] + +permissions: + issues: write + +jobs: + export: + uses: "puppetlabs/phoenix-github-actions/.github/workflows/jira.yml@main" + with: + jira-project: PA + jira-base-url: ${{ vars.jira_base_url }} + jira-user-email: ${{ vars.jira_user_email }} + secrets: + jira-api-token: ${{ secrets.JIRA_ISSUES_ACTION }} diff --git a/.github/workflows/static_code_analysis.yaml b/.github/workflows/static_code_analysis.yaml index 575cde7..62438da 100644 --- a/.github/workflows/static_code_analysis.yaml +++ b/.github/workflows/static_code_analysis.yaml @@ -6,44 +6,8 @@ on: branches: [ main ] pull_request: branches: [ main ] + workflow_dispatch: jobs: static_code_analysis: - name: Run checks - - env: - ruby_version: 2.6 - extra_checks: check:symlinks check:git_ignore check:dot_underscore check:test_file - - runs-on: 'ubuntu-20.04' - steps: - - name: Checkout current PR code - uses: actions/checkout@v4 - with: - fetch-depth: 0 - - - name: Install ruby version ${{ env.ruby_version }} - uses: ruby/setup-ruby@v1 - with: - ruby-version: ${{ env.ruby_version }} - - - name: Prepare testing environment with bundler - run: | - git config --global core.longpaths true - bundle config set --local without 'release' - bundle update --jobs 4 --retry 3 - - - name: Run commits check - run: bundle exec rake commits - - - name: Run rubocop check - run: bundle exec rake ${{ env.extra_checks }} rubocop - - - name: Run syntax check - run: bundle exec rake ${{ env.extra_checks }} syntax syntax:hiera syntax:manifests syntax:templates - - - name: Run lint check - run: bundle exec rake ${{ env.extra_checks }} lint - - - name: Run metadata_lint check - run: bundle exec rake ${{ env.extra_checks }} metadata_lint + uses: "puppetlabs/phoenix-github-actions/.github/workflows/static_code_analysis.yaml@main" diff --git a/metadata.json b/metadata.json index 5306f2a..acbc93d 100644 --- a/metadata.json +++ b/metadata.json @@ -6,7 +6,7 @@ "license": "Apache-2.0", "source": "https://github.com/puppetlabs/puppetlabs-host_core", "project_page": "https://github.com/puppetlabs/puppetlabs-host_core", - "issues_url": "https://tickets.puppetlabs.com/projects/MODULES", + "issues_url": "https://github.com/puppetlabss/puppetlabs-host_core/issues", "dependencies": [ ], diff --git a/rakelib/commits.rake b/rakelib/commits.rake deleted file mode 100644 index 7cee9fa..0000000 --- a/rakelib/commits.rake +++ /dev/null @@ -1,25 +0,0 @@ -desc "verify that commit summaries are properly formatted" -task(:commits) do - # This rake task looks at the summary from every commit from this branch not - # in the branch targeted for a PR. - commit_range = 'HEAD^..HEAD' - puts "Checking commits #{commit_range}" - %x{git log --no-merges --pretty=%s #{commit_range}}.each_line do |commit_summary| - # This regex tests for the currently supported commit summary tokens. - # The exception tries to explain it in more full. - if /^Release prep|\((maint|packaging|doc|docs|modules|pa-\d+)\)|revert/i.match(commit_summary).nil? - raise "\n\n\n\tPlease make sure that your commit summary (i.e. the first line of the commit message) starts with one of the following:\n" \ - "\t\t(PA-<digits>)\n" \ - "\t\t(MODULES-<digits>)\n" \ - "\t\t(docs)\n" \ - "\t\t(docs)(DOCUMENT-<digits>)\n" \ - "\t\t(packaging)\n" - "\t\t(maint)\n" \ - "\t\tRelease prep v<tag>\n" \ - "\n\tThis test for the commit summary is case-insensitive.\n\n\n" - else - puts "#{commit_summary}" - end - puts "...passed" - end -end |