Skip to content

Conversation

@pokey
Copy link
Member

@pokey pokey commented Jan 26, 2022

  • Adds cursorless talon as a subrepo using git-subrepo
  • Pushes sub repo on merge to master
  • Does python linting of the cursorless talon sub repo as part of test

Fixes #475
Fixes #17

push-cursorless-talon:
name: Push cursorless-talon subrepo
runs-on: ubuntu-latest
needs: publish-extension
Copy link
Member Author

Choose a reason for hiding this comment

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

We make sure extension publication was successful before pushing talon side so that users don't end up with new talon files but old extension

steps:
- uses: actions/checkout@v2
with:
fetch-depth: 0
Copy link
Member Author

Choose a reason for hiding this comment

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

Do deep clone so that we can use it for our git push

uses: actions/checkout@v2
with:
repository: cursorless-dev/cursorless-talon
token: ${{ secrets.CURSORLESS_BOT_TOKEN }}
Copy link
Member Author

Choose a reason for hiding this comment

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

We now have a bot 🤖. Only the bot is allowed to push to main on cursorless-talon now

- name: Checkout cursorless-talon
uses: actions/checkout@v2
with:
repository: cursorless-dev/cursorless-talon
Copy link
Member Author

Choose a reason for hiding this comment

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

Notice we moved cursorless-talon into a new cursorless-dev organization, because that's the only way to make it so that only certain users can push to a branch

@@ -0,0 +1 @@
* @pokey No newline at end of file
Copy link
Member Author

Choose a reason for hiding this comment

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

All the files under cursorless-talon just come from the main branch there

@@ -0,0 +1,10 @@
name: Lint
Copy link
Member Author

Choose a reason for hiding this comment

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

Not sure we need to run linting there as well 🤔

@@ -0,0 +1,12 @@
; DO NOT EDIT (unless you know what you are doing)
;
Copy link
Member Author

Choose a reason for hiding this comment

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

So this is the file generated by git-subrepo. Keeps track of all the information needed for pushing. Pretty cool

@pokey pokey merged commit fd7736e into main Jan 27, 2022
@pokey pokey deleted the pokey/issue475-Move-to-monorepo branch January 27, 2022 09:32
pokey added a commit that referenced this pull request Feb 1, 2022
* git subrepo clone (merge) https://github.com/cursorless-dev/cursorless-talon.git cursorless-talon

subrepo:
  subdir:   "cursorless-talon"
  merged:   "afa65b88"
upstream:
  origin:   "https://github.com/cursorless-dev/cursorless-talon.git"
  branch:   "main"
  commit:   "afa65b88"
git-subrepo:
  version:  "0.4.3"
  origin:   "https://github.com/ingydotnet/git-subrepo"
  commit:   "2f68596"

* Set up tests and deploy for cursorless talon subrepo

* Cleaner solution
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 to monorepo Add continuous delivery

3 participants