ElementsProject / ElementsProject/elements

merge script TODOs

Open
#1,411 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
1.2k
Forks
416
Avg merge
1d 23h
Merged PRs (30d)
15

Description

from #923 which was superseded by #1210

evaluate what still needs to be done to improve the script

TODO

- [ ] check whether we're running from /tmp and don't output the "bail out" warning
- [ ] check for chronic and any other tools we might need
- [ ] use ccache rather than skipping autogen/configure (and configure in multiple ways?)
- [ ] don't require running from the project root
- [ ] check for required remotes, required branches, etc; notice if merged-master is out of date
- [ ] notice if there are uncommitted changes and refuse to run
- [ ] add "dry run" mode
- [ ] add mode to validate others' commits, including noticing skipped PRs

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.