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

Revert LFS diff #37

Merged
merged 3 commits into from
Mar 8, 2024
Merged

Revert LFS diff #37

merged 3 commits into from
Mar 8, 2024

Conversation

rzadp
Copy link
Contributor

@rzadp rzadp commented Mar 8, 2024

Reverts #35

It was supposed to change only the diff, not actually store it somewhere else - reverting the whole change for now.

@rzadp rzadp requested a review from a team as a code owner March 8, 2024 07:39
@rzadp rzadp merged commit 2fd06c9 into main Mar 8, 2024
3 checks passed
@rzadp rzadp deleted the rzadp/revert-lfs branch March 8, 2024 07:44
bkchr pushed a commit to polkadot-fellows/RFCs that referenced this pull request Mar 11, 2024
When doing #80 I got confused and updated the wrong action (which uses
the same dependency).

This one updates the RFC action for the cron job

---

When a RFC closes after the last run, it will comment on the open PR (if
it is still open) that the RFC has been closed and how to trigger the
bot to merge the PR.

If it is succesfull, we can automate it to simply close the PRs without
human input.

### Miscellaneous changes

- Also, updated cron job to run at mid day and at end of the day so we
can capture new RFCs more often.
- `v0.1.1` also has a fix for a problem with LFS files not being
downloaded by the GitHub Actions that was discovered and fixed in
paritytech/rfc-action#37 by @rzadp.
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.

1 participant