Skip to content
This repository has been archived by the owner on Feb 1, 2022. It is now read-only.

Update versioning strategy and doc #17

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

tschaffter
Copy link
Member

@tschaffter tschaffter commented Jun 30, 2021

Notes

  • Improve CI workflow using workflow from nlpsandbox/i2b2-phi-dataset.
    • One improvement is that configuration variables are now groups in a workflow job called configure.
  • Remove Docker and GH Pages tags that may never be used in order to simplify the docs

@tschaffter tschaffter self-assigned this Jun 30, 2021
@tschaffter tschaffter added this to Incoming in Research & Benchmarking Technology via automation Jun 30, 2021
@tschaffter tschaffter moved this from Incoming to In progress in Research & Benchmarking Technology Jun 30, 2021
@tschaffter tschaffter changed the title Update versioning ci doc Update versioning strategy and doc Jun 30, 2021
Copy link
Collaborator

@Aryllen Aryllen left a comment

Choose a reason for hiding this comment

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

Only briefly looked this over, but I do have a question. This appears to be running the full build/publish CI on pull request. Would it be better to separate this out into two checks, one that does all the lint/testing for PRs and one that does everything including pushing the new version on push to main (or alternatively just running the publishing part iff on main)?

@tschaffter tschaffter moved this from Documentation 📚 to Tools in Research & Benchmarking Technology Backlog Oct 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Identify versioning protocol Fix link to in README after renaming repo
2 participants