[ENH] Switch datman's CI config to github actions + update package configuration by DESm1th · Pull Request #335 · TIGRLab/datman

DESm1th · 2022-09-19T18:57:04Z

This pull request updates our CI pipelines to use github actions instead of circleci. It also cleans up our existing configuration to remove some clutter.

Add github actions workflows that:
- Automatically label pull requests, and comment on malformed PR titles (this is used later for automated changelogs) (3567059)
- Automatically run tests against python 3.8, 3.9 and 3.10 and post test coverage comment on PR (dea9f51)
- Automatically generate and publish docs to our site (83fb207)
- Automatically generate a release when tags are pushed (generates a changelog, adds changelog to release notes on github, updates zenodo creators list order, and pushes to test-pypi and then pypi) (59c3025)
Refactor the Dockerfile to speed up builds (74d0d2f)
Move all project configuration into pyproject.toml, and remove setup.py and setup.cfg (b7ea1a0, 35faa86, dbd93f1)
Use new importlib.metadata package to version package from git tags, remove versioneer (a27b97d)
Fix some docs config/build issues
- Remove docs/requirements.txt file (is now contained in optional dependencies in pyproject.toml) (41a33f1)
- Move README.rst to the docs folder (3f0b31b)
- Fix build issue from certain links being declared as having duplicate names (7b373d7)
- Remove versioneer references from docs/conf.py (4eb0721)
Update zenodo configuration (8896a94, 9a0ccb2)
- Fix format error in .zenodo.json contributors entries
- Remove .maintenance folder and move update_zenodo.py to .github folder
- Add some missing creators
- Modify update_zenodo.py to use git-summary --line instead of now removed git-line-summary tool
Remove old/unused config files, remove versioneer from manifest.in, rename CHANGES.rst to CHANGELOG.md (name change needed by a github action) (5887bb0)
Tweak auto-assign config to always assign me and randomly choose two others, and to use the github action instead of the old bot (which has sporadically and mysteriously broken on us) (85997ed)
Adds tests, coverage and zenodo badges to our readme (717df18)
Updates contributions guidelines, moves the file to our docs, updates link references to it (b6a343b)

- Unpinned the ubuntu:focal version, since we dont need that much specificity - Set the pythonpath explicitly, for tests and such - Get datman with the copy command instead - Install datman + dependencies last to avoid rebuilding whole container when changes occur

- Add missing 'type' field to contributor entries - Add missing creators - Remove .maintenance folder - Move the update_zenodo.py script to the .github folder

- Changelog updates based on merged PRs + their labels - Zenodo creator list is reorganized based on line contributions - Build is pushed to test-pypi and then pypi, if earlier steps succeed

- Change CONTRIBUTION.md to rst format - Move contributing.rst to the docs folder - Update the newPRBot comment to point to the rendered website page. - Add pylint to style optional dependencies

DESm1th · 2022-09-19T19:00:10Z

The 'add a label' and 'tests' check failures are, I believe, because it's trying to use my credentials to run things owned by tigrlab. They should run properly once the PR is merged.

DESm1th · 2022-09-19T21:29:18Z

Oh! Also, @jerdra , @josephmje , @salimansourc do you guys want to be added to the maintainer list? If so what email should I throw on?

@all-contributors add @josephmje , @jerdra, @salimansourc for code, maintenance, review

allcontributors · 2022-09-19T21:29:26Z

@DESm1th

I've put up a pull request to add @josephmje! 🎉

DESm1th · 2022-09-19T21:32:13Z

@all-contributors add @jerdra for code, maintenance, review

allcontributors · 2022-09-19T21:32:23Z

@DESm1th

I've put up a pull request to add @jerdra! 🎉

DESm1th · 2022-09-19T21:32:40Z

@all-contributors add @salimansourc for code, maintenance, review

allcontributors · 2022-09-19T21:32:48Z

@DESm1th

I've put up a pull request to add @salimansourc! 🎉

josephmje

Thanks for this @DESm1th! Excited about the new Github actions you've added here, this automates so much!

I just left a few very minor comments

Co-authored-by: Michael Joseph <josephmje.22@gmail.com>

DESm1th · 2022-09-22T18:26:40Z

@all-contributors add @slimnsour for code, maintenance, review

allcontributors · 2022-09-22T18:26:48Z

@DESm1th

I've put up a pull request to add @slimnsour! 🎉

DESm1th added 27 commits September 9, 2022 20:17

[ENH] Add flake8 config

b7ea1a0

[ENH] Add pytest config to pyproject.toml

35faa86

[REF] Remove unneeded setup files

dbd93f1

[REF] Remove unneeded docs/requirements.txt file

41a33f1

[REF] Update version management to be simpler

a27b97d

[FIX] Update docs config to remove versioneer references

4eb0721

[FIX] Change links to anonymous reference to stop rst build errors

7b373d7

[REF] Move the README.rst to the docs folder

3f0b31b

[FIX] Update .zenodo.json, move update script

8896a94

- Add missing 'type' field to contributor entries - Add missing creators - Remove .maintenance folder - Move the update_zenodo.py script to the .github folder

[FIX] Removed un-needed files, updated manifest, renamed change log

5887bb0

[FIX] Update changelog header format for github action

daa0cfe

[FIX] Update update_zenodo.py script to use git-summary instead

9a0ccb2

[ENH] Update auto-assign configuration

85997ed

[ENH] Add automated PR labeling based on title

3567059

[ENH] Cancel earlier runs of labeling workflow if multiple overlap

5233941

[ENH] Add test workflow

dea9f51

[FIX] Bump python version + loosen requirements for numpy/pandas

9c73015

[FIX] Add config file for PR label assignment

c5397ac

[ENH] Add docs update workflow

83fb207

[FIX] Update classifiers with new supported python versions

7a57031

[ENH] Add workflow to make release

59c3025

- Changelog updates based on merged PRs + their labels - Zenodo creator list is reorganized based on line contributions - Build is pushed to test-pypi and then pypi, if earlier steps succeed

[REF] Rename the release workflow

6c03dbb

[FIX] Update reference to change log file

d8f96b5

[ENH] Add settings for pylint (disable log format interpolation)

43e1e12

[ENH] Add badges to readme, and link to docs

717df18

[DOC] Update the contribution guidelines

b6a343b

- Change CONTRIBUTION.md to rst format - Move contributing.rst to the docs folder - Update the newPRBot comment to point to the rendered website page. - Add pylint to style optional dependencies

DESm1th requested review from jerdra and josephmje September 19, 2022 18:57

DESm1th self-assigned this Sep 19, 2022

auto-assign bot requested a review from benselby September 19, 2022 18:57

TIGRLab deleted a comment from allcontributors bot Sep 19, 2022

allcontributors bot mentioned this pull request Sep 19, 2022

add josephmje as a contributor #336

Merged

TIGRLab deleted a comment from allcontributors bot Sep 19, 2022

allcontributors bot mentioned this pull request Sep 19, 2022

add jerdra as a contributor #337

Merged

allcontributors bot mentioned this pull request Sep 19, 2022

add salimansourc as a contributor #338

Closed

josephmje previously approved these changes Sep 22, 2022

View reviewed changes

Comment thread .github/comment_pr_title/action.yml Outdated

Comment thread .github/auto_assign.yml

Comment thread .github/labeler.yml Outdated

Update .github/labeler.yml

7683dce

Co-authored-by: Michael Joseph <josephmje.22@gmail.com>

DESm1th dismissed josephmje’s stale review via 7683dce September 22, 2022 18:10

DESm1th added 2 commits September 22, 2022 14:19

[ENH] Add Mike's suggestion to allow 'enh:' as a valid prefix

02aa8f4

[FIX] Add salim to the auto assign list

79adb0c

allcontributors bot mentioned this pull request Sep 22, 2022

add slimnsour as a contributor #339

Merged

DESm1th merged commit 32d9b6c into TIGRLab:main Oct 6, 2022

DESm1th deleted the config_updates branch October 6, 2022 18:14

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[ENH] Switch datman's CI config to github actions + update package configuration#335

[ENH] Switch datman's CI config to github actions + update package configuration#335
DESm1th merged 30 commits intoTIGRLab:mainfrom
DESm1th:config_updates

DESm1th commented Sep 19, 2022

Uh oh!

DESm1th commented Sep 19, 2022

Uh oh!

DESm1th commented Sep 19, 2022

Uh oh!

allcontributors bot commented Sep 19, 2022

Uh oh!

DESm1th commented Sep 19, 2022

Uh oh!

allcontributors bot commented Sep 19, 2022

Uh oh!

DESm1th commented Sep 19, 2022

Uh oh!

allcontributors bot commented Sep 19, 2022

Uh oh!

josephmje left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

DESm1th commented Sep 22, 2022

Uh oh!

allcontributors bot commented Sep 22, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

DESm1th commented Sep 19, 2022

Uh oh!

DESm1th commented Sep 19, 2022

Uh oh!

DESm1th commented Sep 19, 2022

Uh oh!

allcontributors bot commented Sep 19, 2022

Uh oh!

DESm1th commented Sep 19, 2022

Uh oh!

allcontributors bot commented Sep 19, 2022

Uh oh!

DESm1th commented Sep 19, 2022

Uh oh!

allcontributors bot commented Sep 19, 2022

Uh oh!

josephmje left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

DESm1th commented Sep 22, 2022

Uh oh!

allcontributors bot commented Sep 22, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants