search

exercism / org-wide-files

MIT License
7 stars 8 forks source link

[Tracking] Sync Tooling - push - ErikSchierboom - cc4bf900aaee77af9629a751dadc8092d82e379a #154

Closed github-actions[bot] closed 2 years ago

github-actions[bot] commented 2 years ago

Workflow run: https://github.com/exercism/org-wide-files/actions/runs/1933792482

verdammelt commented 2 years ago

@ErikSchierboom I'm curious that while my track's (common-lisp) analyzer and representer were updated, the test runner was not. That repo does have a sync-labels script so it should have gotten an update right?

SaschaMann commented 2 years ago

@verdammelt we ran into secondary rate limits (which seem to be stricter than they used to), the script hasn't gotten to common-lisp-test-runner yet. It'll be done next week when Erik is back.

ynfle commented 2 years ago

@ErikSchierboom https://github.com/exercism/nim-docker-base was left out

ynfle commented 2 years ago

Actually, I'm not sure. We don't need to push to AWS, so we have the following lines added to deploy.yml:

with:
  aws_ecr: false
ErikSchierboom commented 2 years ago

@ynfle There were some rate limit issues.

ynfle commented 2 years ago

Ya realized after I made the comment. Not sure we want to be part of this now

ErikSchierboom commented 2 years ago

@ynfle I could remove the exercism-tooling tag, which would opt out of the deploy stuff.

ynfle commented 2 years ago

I think that'd be the right way to go.

We could just push to AWS also, but it isn't needed. What do you think?

ErikSchierboom commented 2 years ago

I could see arguments both ways, but I'm fine with removing the label.

ErikSchierboom commented 2 years ago

Done