Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@revi
Copy link
Contributor

@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.

@revi revi mentioned this pull request Dec 30, 2024
5 tasks
@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
Contributor 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