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

Add GitHub Actions #275

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Add GitHub Actions #275

wants to merge 2 commits into from

Conversation

smyrman
Copy link
Collaborator

@smyrman smyrman commented Oct 1, 2021

Back-ported from https://github.com/searis/rest-layer

Badges in README not updated.

@smyrman smyrman requested a review from rs October 1, 2021 07:11
@smyrman
Copy link
Collaborator Author

smyrman commented Oct 1, 2021

If you want to add GitHub actions to this repo, here is a starting-point. If you enable GitHub Actions actions for the repo, it will, run, but at least the golang-ci linter task is going to fail.

Remove test from back-ports for non-existent folder storers/mongo.
@smyrman
Copy link
Collaborator Author

smyrman commented Oct 1, 2021

As mentioned on Slack, we at Clarify will be maintaining our own fork of rest-layer going forward; it just makes the development faster and easier for us to control. I have got all the tests working there, and fixed all the linter issues, but we haven't yet transitioned to using it.

We are very grateful for rest-layer, and will continue to maintain the fork with bugfixes and minor features.

@rs
Copy link
Owner

rs commented Oct 1, 2021

Why not moving the main project into an org?

@smyrman
Copy link
Collaborator Author

smyrman commented Oct 2, 2021

Why not moving the main project into an org?

It does make sense to discuss this properly. I started a discussion here clarify#2. It's our "selfish" view of what we need from rest-layer going forward, and what we hope to achieve with a fork. Contributions and views are very much welcome.

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.

2 participants