Skip to content
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

migrates to github actions #52

Merged
merged 7 commits into from
Mar 14, 2022

Conversation

simonneutert
Copy link
Collaborator

@simonneutert simonneutert commented Mar 12, 2022

i hope this is what you'd wanted to see, copied the key from the link you provided.

https://github.com/slack-ruby/slack-ruby-bot-server/pull/142/files

closes #42

Copy link
Owner

@dblock dblock left a comment

Choose a reason for hiding this comment

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

I see https://github.com/simonneutert/strava-ruby-client/runs/5522237082?check_suite_focus=true has failures, do you have a green pass?

I think you need to lock ActiveSupport to <= 6 and we can fix 7 later.

@simonneutert
Copy link
Collaborator Author

please verify my changes for active_support I do not fully understand why rspec is having a hard time

Copy link
Owner

@dblock dblock left a comment

Choose a reason for hiding this comment

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

Use require 'active_support', remove comments. Almost there.

@dblock
Copy link
Owner

dblock commented Mar 12, 2022

Oh and add a CHANGELOG entry for this one.

@simonneutert
Copy link
Collaborator Author

I cleaned up the code @dblock

@simonneutert
Copy link
Collaborator Author

I cleaned up the code @dblock

line 14 in danger.yml errors

@simonneutert
Copy link
Collaborator Author

I hope this is working fine now, else you might need to get your hands dirty, sorry 😞

@dblock dblock merged commit f5757f5 into dblock:master Mar 14, 2022
@dblock
Copy link
Owner

dblock commented Mar 14, 2022

I merged it, will cleanup next.

@simonneutert simonneutert deleted the 42-migrate-togh-actions branch September 17, 2024 09:16
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.

Update Travis Badge or migrate to GHA
2 participants