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.

Sign up for GitHub

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

Jump to bottom

PR just for CI #1

Open
wants to merge 48 commits into
base: main
Choose a base branch
from
Open

PR just for CI #1

wants to merge 48 commits into from

Conversation

coreyja
Copy link
Owner

@coreyja coreyja commented Mar 15, 2023

No description provided.

coreyja and others added 30 commits February 25, 2023 17:47
@coreyja
Create a new binary package for cargo-typify
a127a20 
This creates a new binary package to use and adds it to the workspace.

Unfortunately there doesn't seem like a good way to make it the default
run command for the entire workspace.
See this issue for more details: rust-lang/cargo#7290
@coreyja
First version
a8d1ced 
This is a really bare bones version, but its working

We use `clap` for Arg parsing, and `color_eyre` for error handling.

We take an `input` arg and an optional `output` arg that both take a path to a file.
If no output is specified, we print to stdout.
@coreyja
Just cleanup the Cargo.toml file a bit
6078576
@coreyja
Unify all the schemars versions across the workspace
cc49c4f
@coreyja
Cleanup unused import and format Cargo.toml
47f3074
@coreyja
Remove this example schema since I can use the ones in other dirs
2c89d5d
@dependabot
Bump syn from 1.0.107 to 1.0.109 (oxidecomputer#206)
141be90 
Bumps [syn](https://github.com/dtolnay/syn) from 1.0.107 to 1.0.109.
- [Release notes](https://github.com/dtolnay/syn/releases)
- [Commits](dtolnay/syn@1.0.107...1.0.109)

---
updated-dependencies:
- dependency-name: syn
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump schemars from 0.8.11 to 0.8.12 (oxidecomputer#205)
05d65ea 
Bumps [schemars](https://github.com/GREsau/schemars) from 0.8.11 to 0.8.12.
- [Release notes](https://github.com/GREsau/schemars/releases)
- [Changelog](https://github.com/GREsau/schemars/blob/master/CHANGELOG.md)
- [Commits](GREsau/schemars@v0.8.11...v0.8.12)

---
updated-dependencies:
- dependency-name: schemars
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump thiserror from 1.0.38 to 1.0.39 (oxidecomputer#212)
97e149b 
Bumps [thiserror](https://github.com/dtolnay/thiserror) from 1.0.38 to 1.0.39.
- [Release notes](https://github.com/dtolnay/thiserror/releases)
- [Commits](dtolnay/thiserror@1.0.38...1.0.39)

---
updated-dependencies:
- dependency-name: thiserror
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump paste from 1.0.11 to 1.0.12 (oxidecomputer#211)
25f4293 
Bumps [paste](https://github.com/dtolnay/paste) from 1.0.11 to 1.0.12.
- [Release notes](https://github.com/dtolnay/paste/releases)
- [Commits](dtolnay/paste@1.0.11...1.0.12)

---
updated-dependencies:
- dependency-name: paste
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump serde_json from 1.0.93 to 1.0.94 (oxidecomputer#210)
be916d5 
Bumps [serde_json](https://github.com/serde-rs/json) from 1.0.93 to 1.0.94.
- [Release notes](https://github.com/serde-rs/json/releases)
- [Commits](serde-rs/json@v1.0.93...v1.0.94)

---
updated-dependencies:
- dependency-name: serde_json
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump unicode-ident from 1.0.6 to 1.0.8 (oxidecomputer#208)
68872f0 
Bumps [unicode-ident](https://github.com/dtolnay/unicode-ident) from 1.0.6 to 1.0.8.
- [Release notes](https://github.com/dtolnay/unicode-ident/releases)
- [Commits](dtolnay/unicode-ident@1.0.6...1.0.8)

---
updated-dependencies:
- dependency-name: unicode-ident
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump trybuild from 1.0.77 to 1.0.79 (oxidecomputer#209)
e01e314 
Bumps [trybuild](https://github.com/dtolnay/trybuild) from 1.0.77 to 1.0.79.
- [Release notes](https://github.com/dtolnay/trybuild/releases)
- [Commits](dtolnay/trybuild@1.0.77...1.0.79)

---
updated-dependencies:
- dependency-name: trybuild
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@jayvdb
make dependency on typify-macro optional (oxidecomputer#213)
18e7faf
@jayvdb
fix spelling (oxidecomputer#214)
d1cae0f
@coreyja
Revert making schemars a workspace dep, that change should be in its …
bcc29d5 
…own PR
@coreyja
Copy release.toml from typify-impl
c20f23b
@coreyja
Add the clippy bits at the beggining
d6202e6
@coreyja
Take the suggested doc comment, which will be the help text in the CLI
0a1a267
@coreyja
Use Expectorate to setup a basic test for converting a file
42e1aa2
@coreyja
Lets run rust_fmt on the input
74c0e19
@coreyja
Extract to a lib.rs and do tests in there
cced055
@coreyja
Make the input path to the cli a positional instead of an arg
abde9fc
@coreyja
Add arg to create builder interface and add test for it
a726932
@coreyja
Add integration tests but they are really the same as the other tests…
26247ab 
… so maybe we only need the integration versions?
@coreyja
Test the stdout as well
851adc7
@coreyja
Test a single additional derive
accbf7e
@coreyja
Get multiple derives working as well
5e06e97
@coreyja
Get something commited for type_mod but its not working
c278744
@coreyja
Add default run
71c2b40
coreyja and others added 18 commits March 10, 2023 21:16
@coreyja
Merge branch 'main' into ca/main/cargo-typify
ba40e53 
* main:
  fix spelling (oxidecomputer#214)
  make dependency on typify-macro optional (oxidecomputer#213)
  Bump trybuild from 1.0.77 to 1.0.79 (oxidecomputer#209)
  Bump unicode-ident from 1.0.6 to 1.0.8 (oxidecomputer#208)
  Bump serde_json from 1.0.93 to 1.0.94 (oxidecomputer#210)
  Bump paste from 1.0.11 to 1.0.12 (oxidecomputer#211)
  Bump thiserror from 1.0.38 to 1.0.39 (oxidecomputer#212)
  Bump schemars from 0.8.11 to 0.8.12 (oxidecomputer#205)
  Bump syn from 1.0.107 to 1.0.109 (oxidecomputer#206)
@coreyja
Remove this todo since it doesn't change the output
ffe8a4e
@coreyja
Use the same version of schemars as the other crates
407504d
@coreyja
We aren't actually using this one
2dba827
@dependabot
Bump regress from 0.4.1 to 0.5.0 (oxidecomputer#216)
e737a4a 
Bumps [regress](https://github.com/ridiculousfish/regress) from 0.4.1 to 0.5.0.
- [Release notes](https://github.com/ridiculousfish/regress/releases)
- [Commits](ridiculousfish/regress@v0.4.1...v0.5.0)

---
updated-dependencies:
- dependency-name: regress
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump serde_tokenstream from 0.1.6 to 0.1.7 (oxidecomputer#218)
7cd107a 
Bumps [serde_tokenstream](https://github.com/oxidecomputer/serde_tokenstream) from 0.1.6 to 0.1.7.
- [Release notes](https://github.com/oxidecomputer/serde_tokenstream/releases)
- [Commits](oxidecomputer/serde_tokenstream@v0.1.6...v0.1.7)

---
updated-dependencies:
- dependency-name: serde_tokenstream
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump quote from 1.0.23 to 1.0.24 (oxidecomputer#220)
3585dd3 
Bumps [quote](https://github.com/dtolnay/quote) from 1.0.23 to 1.0.24.
- [Release notes](https://github.com/dtolnay/quote/releases)
- [Commits](dtolnay/quote@1.0.23...1.0.24)

---
updated-dependencies:
- dependency-name: quote
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump proc-macro2 from 1.0.51 to 1.0.52 (oxidecomputer#219)
08f20a6 
Bumps [proc-macro2](https://github.com/dtolnay/proc-macro2) from 1.0.51 to 1.0.52.
- [Release notes](https://github.com/dtolnay/proc-macro2/releases)
- [Commits](dtolnay/proc-macro2@1.0.51...1.0.52)

---
updated-dependencies:
- dependency-name: proc-macro2
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump serde from 1.0.152 to 1.0.155 (oxidecomputer#217)
945d551 
Bumps [serde](https://github.com/serde-rs/serde) from 1.0.152 to 1.0.155.
- [Release notes](https://github.com/serde-rs/serde/releases)
- [Commits](serde-rs/serde@v1.0.152...v1.0.155)

---
updated-dependencies:
- dependency-name: serde
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@jayvdb
expand docs for TypeSpaceSettings (oxidecomputer#202)
bf60070
@coreyja
Add test for the help output, and remove predicates dep since we can …
497e788 
…do full matches not contains
@coreyja
Change the CLI so that we do a replace of the file with a new extensi…
042ff53 
…on by default. You can opt into either a file or stdout
@coreyja
Remove type_mod that wasn't doing anything to the output
6955c0e
@coreyja
Rework this to work as a cargo sub-command
cddde14
@coreyja
Remove non-integration specs and add some simple specs for the parsin…
c457b6a 
…g of the output path
@coreyja
Merge branch 'main' into ca/main/cargo-typify
b0f93e0 
# By dependabot[bot] (5) and John Vandenberg (1)
# Via GitHub
* main:
  expand docs for TypeSpaceSettings (oxidecomputer#202)
  Bump serde from 1.0.152 to 1.0.155 (oxidecomputer#217)
  Bump proc-macro2 from 1.0.51 to 1.0.52 (oxidecomputer#219)
  Bump quote from 1.0.23 to 1.0.24 (oxidecomputer#220)
  Bump serde_tokenstream from 0.1.6 to 0.1.7 (oxidecomputer#218)
  Bump regress from 0.4.1 to 0.5.0 (oxidecomputer#216)

# Conflicts:
#	Cargo.lock
@coreyja
Add positional arg as inverse of builder, builder is now the default
95241d2
@coreyja
Convert line endings for tests in Windows
84491f6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@coreyja @jayvdb