summaryrefslogtreecommitdiff
path: root/maint
Commit message (Collapse)AuthorAgeFilesLines
...
* clippy: globally allow clippy::needless_raw_string_hashesIan Jackson2023-07-101-0/+1
| | | | | | | | | | | | Rationale: no-one writes these by default without thinkinh. If they are unnecessary, then either the string must have had " in it before (in which case it might do again), or it is near other strings which *do* need it. And having it does no harm; indeed IMO it can increase clarity. Alternative to !1388's Fix new "needless_raw_string_hashes" lint from clippy +nightly
* Merge branch 'allow_missing_panics' into 'main'Ian Jackson2023-07-071-1/+0
|\ | | | | | | | | | | | | add_warning: Tolerate clippy::missing_panics_doc Closes #950 See merge request tpo/core/arti!1380
| * add_warning: Tolerate clippy::missing_panics_docNick Mathewson2023-07-061-1/+0
| | | | | | | | | | | | | | As of current nightly, this lint now includes expect()s, which makes it trigger way more than we'd like. See discussion on #950.
* | tor-rtcompat: Simplify trivial all() expressions.Nick Mathewson2023-07-061-2/+2
|/ | | | | | | Clippy nightly doesn't like `#[cfg(all(...))]` with only a single expression inside the `all(...)`. This requires an adjustment in check_doc_features.
* Replace our old complex semver-checks script.Nick Mathewson2023-06-291-106/+6
| | | | | | | | Now with cargo semver-checks >= 0.22.1, we no longer need to jump through hoops in order to only look at the `full` features. This, combined with our work on `fixup-features`, lets us be confident that we're only looking at semver breakage in experimental code.
* lints: Promote clippy::print_stderr and clippy::print_stdoutIan Jackson2023-06-211-0/+2
| | | | | | | | These are available in our MSRV now, so we don't need to handle specially. We can just add them to the standard lint block. (Lint block in every crate will be updated automatically in the next commit.)
* CI: add script to test every crate with default featuresIan Jackson2023-06-141-0/+19
|
* maint/cargo-audit: Ignore RUSTSEC-2023-0040 (the users crate is unmaintained)Gabriela Moldovan2023-06-051-0/+3
| | | | Signed-off-by: Gabriela Moldovan <[email protected]>
* New script to run cargo-semver-checks with appropriate options.Nick Mathewson2023-05-311-0/+121
| | | | | | | | | | | | | The options are rather complicated; because we do not want to subject our experimental features to semver, we need to run generate JSON rustdoc on our own and then pass that JSON to cargo-semver-checks. This in turn requires us to use the same options that cargo-semver-checks uses, including "RUSTC_BOOTSTRAP". I've left some TODOs here in places where we will likely want to improve our code in the future. See #711.
* maint/check_license: Make MPL-2.0 into an allow-listNick Mathewson2023-05-231-1/+22
| | | | | | | | | | | | | Previously we allowed this license unconditionally. But because of its non-self-enacting nature, we need the actual notice from its "exhibit A" to appear somewhere that says that it applies to all the relevant code. Therefore, we shouldn't take new MPL-2.0 dependencies without hand-checking them. (I am tentatively allowing option-ext, though, since we already have an indirect dependency on that crate via `directories`.) For more info, see https://gitlab.torproject.org/tpo/core/arti/-/issues/845
* thanks: Also acknowledge Suggested-ByNick Mathewson2023-05-221-0/+2
|
* maint/thanks: Split up some long pipelinesNick Mathewson2023-05-181-2/+9
|
* maint/thanks: Remove email addresses from git trailersNick Mathewson2023-05-181-1/+1
| | | | | Okay, technically we're removing everything between the first `<` and the `>` at the end of the line.
* maint/thanks: Include some git trailers in acknowledgmentsNick Mathewson2023-05-181-3/+18
| | | | | | | | When building our list of acknowledgments, previously we would only include author and committer names. Now we also include anybody listed in the "Reported-by", "Co-authored-by", and "Thanks" trailers.
* fixup-features: minor doc fix.Nick Mathewson2023-05-151-5/+5
|
* fixup-features: Do not annotate non-features.Nick Mathewson2023-05-152-3/+7
|
* fixup-features: Do not add edges from non-features.Nick Mathewson2023-05-152-2/+7
|
* fixup-features: distinguish internal and external edgesNick Mathewson2023-05-152-3/+8
| | | | An external edge does not cause its target to be created as a feature.
* fixup-features: Add an option to not annotate.Nick Mathewson2023-05-153-6/+25
|
* fixup-features: ability to add annotations for everything.Nick Mathewson2023-05-151-7/+13
|
* fixup-features: fix off-by-one in argument reading.Nick Mathewson2023-05-151-1/+1
|
* fixup-features: Implement remaining rules.Nick Mathewson2023-05-151-34/+64
|
* fixup-features: Enforce __is_experimental tagging rule.Nick Mathewson2023-05-151-0/+19
|
* fixup-features: Revise our rule 2 enforcement to use newer APIs.Nick Mathewson2023-05-151-14/+14
|
* fixup-features: Refactor "apply a list of changes" code into a new module.Nick Mathewson2023-05-152-25/+88
|
* fixup-features: Make a feature graph type in a submoduleNick Mathewson2023-05-153-60/+149
| | | | | I tried to use petgraph, but it was optimized for performance over usability, and the usability was beyond me.
* fixup-features: minor spelling and comment fixes.Nick Mathewson2023-05-151-1/+1
|
* fixup-features: Describe the semantics we actually wantNick Mathewson2023-05-151-6/+26
| | | | | | | | | The problem with our old rules is that "reachable from __nonadditive" and "reachable from experimental" were not themselves sensible definitions of nonadditive and experimental. See https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/1068#note_2887939
* allow rustls-webpki licensetrinity-1686a2023-05-081-0/+1
|
* check-licenses: Permit MPL-2.0Nick Mathewson2023-05-041-0/+1
| | | | | | | | The generational-arena crate is distributed under MPL-2.0, so we need to allow it. I believe that this license is fine and does not interfere with our code or our users; the reviewer should double-check.
* fix macos reproducible buildstrinity-1686a2023-04-161-1/+1
|
* Bump Rust version in reproducible build scriptsNick Mathewson2023-04-131-1/+1
| | | | Now that we require 1.65, Rust 1.63 is no longer sufficient.
* Increment MSRV to 1.65 in every crate.Nick Mathewson2023-04-111-1/+1
|
* Add Gabi to exclude_contributors.Nick Mathewson2023-03-311-0/+3
| | | | (We don't include Tor employees in our "thanks" section.)
* Fix typosDimitris Apostolou2023-03-241-1/+1
|
* maint/check_todos: Offer advice on workarounds, if trippedIan Jackson2023-03-211-0/+22
|
* Obscure various literal uses of the todo markerIan Jackson2023-03-212-2/+2
| | | | In each case, use a circumlocution.
* maint/check_todos: New scriptIan Jackson2023-03-211-0/+31
|
* fixup-features: Use write-and-rename pattern.Nick Mathewson2023-03-201-1/+3
|
* fixup-features: Improve a comment.Nick Mathewson2023-03-201-1/+1
|
* fixup-features: Simplify "optionality" detection.Nick Mathewson2023-03-201-17/+13
|
* New "fixup-features" tool to enforce properties in Cargo.tomlsNick Mathewson2023-03-203-0/+346
| | | | | | See top-level main.rs comment for an explanation. Closes #771
* Merge branch 'check_env' into 'main'Ian Jackson2023-03-012-23/+122
|\ | | | | | | | | maint: add a full dependency check See merge request tpo/core/arti!1030
| * maint: remove the dependency check from coverageEmil Engler2023-02-241-23/+0
| | | | | | | | | | This commit removes the dependency check introduced by b4b8d040, because it's task is now done by the recently added `check_env` script.
| * maint: add a full dependency checkEmil Engler2023-02-241-0/+122
| | | | | | | | | | | | | | This commit adds a script that checks the environment for it's suitable as an Arti development environment. It follows an idea pitched by Ian Jackson and me in !1025.
* | Merge branch 'zlib' into 'main'Ian Jackson2023-02-271-2/+9
|\ \ | | | | | | | | | | | | Approve zlib licence See merge request tpo/core/arti!1035
| * | maint/check_licensse: Placate shellcheckIan Jackson2023-02-241-1/+1
| | | | | | | | | | | | | | | There's nothing wrong with the previous idiom since it's a trusted variable, but whatever.
| * | maint/check_licensse: Approve Zlib licenceIan Jackson2023-02-241-0/+1
| | | | | | | | | | | | | | | | | | | | | This is a very permissive BSD/MIT-like licence. I have reviewed the licence text and it is inoffensive. I want this for slotmap, in !1034
| * | maint/check_licensse: Honour CARGO variableIan Jackson2023-02-241-2/+8
| | | | | | | | | | | | Even if it contains spaces.
| * | maint/check_licensse: Do not discard actual errorIan Jackson2023-02-241-1/+1
| | | | | | | | | | | | This confused me while I was debugging the interaction with nailing-cargo.
generated by cgit v1.2.3 (git 2.46.0) at 2026-08-27 23:35:59 +0000