Skip to content

Conversation

revi
Copy link

@revi revi commented Dec 30, 2024

Those are used by (Phabricator/Phorge)'s arcanist CLI tools.

  • .arcconfig: docs
  • .arclint and .arcunit: docs says "inside .arcconfig" but apparently (according to upstream repo) .arclint and .arcunit is supported as well.
  • Added a change log entry in changelog.d/<directory>/.
  • Added self to copyright blurb of touched files.
  • Added self to AUTHORS.rst.
  • My changes do not contradict
    the current specification.
  • I agree to license my contribution under the licenses indicated in the
    changed files.

@floriansnow
Copy link
Contributor

Thank you for this contribution! Once the conflicts are resolved, we can go ahead and merge this.

Those are used by (Phabricator/Phorge)'s arcanist CLI tools.

- `.arcconfig`: [docs](https://we.phorge.it/book/phorge/article/arcanist_new_project/)
- `.arclint` and `.arcunit`: [docs](https://we.phorge.it/book/phorge/article/arcanist_lint_unit/)
  says "inside `.arcconfig`" but apparently (according to upstream repo) `.arclint`
  and `.arcunit` is supported as well.

Signed-off-by: Yongmin Hong <[email protected]>
@revi
Copy link
Author

revi commented Jul 3, 2025

Rebased with current main branch. (Also rebased #1124 just in case ™️.)

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