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

Fix / Replace Travis CI #1161

Closed
ndanielsen opened this issue Feb 25, 2021 · 6 comments
Closed

Fix / Replace Travis CI #1161

ndanielsen opened this issue Feb 25, 2021 · 6 comments
Labels
priority: high should be done before next release type: technical debt work to optimize or generalize code

Comments

@ndanielsen
Copy link
Contributor

Describe the issue

Looks like Travis CI is no longer functional on this project (and most open source projects). Creating an issue to track the fix.

I put up a work-in-progress issue to replace Travis CI with github actions. I'm happy to complete that if there is any interest in migrating to github actions.

#1158

@DistrictDataLabs/team-oz-maintainers

@ndanielsen
Copy link
Contributor Author

ndanielsen commented Mar 11, 2021

Main body of work merged in: #1158

A few remaining TODOs come to mind:

  • enabling github actions on this repo
  • setting up the slack token access
  • setting up codecov
  • statuses badges for github actions and codecov
  • documentation scan to update any references to coveralls / travis

@ndanielsen
Copy link
Contributor Author

@pdamodaran did you want to pick up the remaining items or should I?

@pdamodaran
Copy link
Contributor

@ndanielsen - thanks for offering to take on additional work in relation to Github Actions! I am new to this so I appreciate the help. Please see my responses below your TODO list.

Main body of work merged in: #1158

A few remaining TODOs come to mind:

  • enabling github actions on this repo
    - I had thought the merge would trigger Github Actions but it didn't so if you can take care of this, it will be helpful.
  • setting up the slack token access
    - this is done
  • setting up codecov
    - this is done
  • statuses badges for github actions and codecov
    - I am curious what this is - and yes would appreciate it if you can take this on as well
  • documentation scan to update any references to coveralls / travis
    - would love the help on this also

@ndanielsen
Copy link
Contributor Author

Just had a moment to look at this now. Seems like you took on the badge item in a PR this morning.

As for the slack token, I can't see the slack token that was set but it needs to be the entire slack_webhook url - not just the token as it was before. I'll let you fix this as I've been avoiding slack in-general for awhile now.

If anything else needs to be done, please loop me in via github.

I'll addresses the minor issue on my pending PR soon.

bbengfort pushed a commit that referenced this issue Apr 2, 2021
Co-authored-by: Prema Roman <premaroman@Blade.local>
Co-authored-by: Prema Roman <premaroman@Blade.hsd1.va.comcast.net>
Co-authored-by: Prema Roman <premaroman@Blade.fios-router.home>
Co-authored-by: Prema Roman <prema.roman@excella.com>
@bbengfort
Copy link
Member

@pdamodaran @ndanielsen it looks like we're all in on GitHub actions now; can we close this issue?

@bbengfort bbengfort added priority: high should be done before next release type: technical debt work to optimize or generalize code labels Apr 2, 2021
@pdamodaran
Copy link
Contributor

@bbengfort - yes I will go ahead and close this issue - thanks for merging the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: high should be done before next release type: technical debt work to optimize or generalize code
Projects
None yet
Development

No branches or pull requests

3 participants