Skip to content

Commit

Permalink
sync-shared-config: fix style and style checks.
Browse files Browse the repository at this point in the history
- Use `exclude?` instead of `include?` to make RuboCop happy
- Properly check the `sync` jobs success so that auto-merge is not
  run if jobs are failing
  • Loading branch information
MikeMcQuaid committed Jun 11, 2024
1 parent 05db2e1 commit 761d4f7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/actions/sync/shared-config.rb
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ def git(*args)
end

# Update Gemfile.lock if it exists, based on the Ruby version.
if !custom_ruby_version_repos.include?(repository_name) && (target_directory_path/"Gemfile.lock").exist?
if custom_ruby_version_repos.exclude?(repository_name) && (target_directory_path/"Gemfile.lock").exist?
Dir.chdir target_directory_path do
puts "Running bundle install..."
system "bundle update --ruby --bundler --quiet >/dev/null"
Expand Down
5 changes: 3 additions & 2 deletions .github/workflows/sync-shared-config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,7 @@ jobs:
conclusion:
needs: sync
runs-on: ubuntu-latest
if: always()
steps:
- name: Conclusion
run: echo "Sync shared configurations completed"
- name: Result
run: ${{ needs.sync.result == 'success' }}

0 comments on commit 761d4f7

Please sign in to comment.