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

Github actions cutover and linting #112

Merged
merged 28 commits into from May 11, 2022
Merged

Github actions cutover and linting #112

merged 28 commits into from May 11, 2022

Conversation

chayim
Copy link
Contributor

@chayim chayim commented Apr 26, 2022

closes #80

@chayim
Copy link
Contributor Author

chayim commented Apr 26, 2022

Sample build: now pushing to s3 snapshots for branchs: https://github.com/RedisGears/rgsync/actions/runs/2225442318
Latest changes re-enable everything.
Note: This PR must be merged after #111 and #58

@chayim chayim marked this pull request as ready for review April 26, 2022 08:59
@lgtm-com
Copy link

lgtm-com bot commented Apr 26, 2022

This pull request introduces 4 alerts and fixes 4 when merging 21dd461 into 7861d51 - view on LGTM.com

new alerts:

  • 3 for 'import *' may pollute namespace
  • 1 for Variable defined multiple times

fixed alerts:

  • 3 for 'import *' may pollute namespace
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Apr 26, 2022

This pull request introduces 4 alerts and fixes 4 when merging ea6d5e2 into 7861d51 - view on LGTM.com

new alerts:

  • 3 for 'import *' may pollute namespace
  • 1 for Variable defined multiple times

fixed alerts:

  • 3 for 'import *' may pollute namespace
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Apr 26, 2022

This pull request introduces 4 alerts and fixes 4 when merging 9688c62 into 7861d51 - view on LGTM.com

new alerts:

  • 3 for 'import *' may pollute namespace
  • 1 for Variable defined multiple times

fixed alerts:

  • 3 for 'import *' may pollute namespace
  • 1 for Unused import

@chayim chayim changed the title Github actions cutover Github actions cutover and linting Apr 26, 2022
@chayim chayim merged commit 0b57e82 into master May 11, 2022
@chayim chayim deleted the ck-byecircle branch May 11, 2022 11:14
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.

Move all CI bits from Circle to GH actions
2 participants