8000 88 support j core by kevinacahalan · Pull Request #108 · SWIM-ucf/SWIM · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

88 support j core #108

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.

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

Merged
kevinacahalan merged 13 commits into from
Jan 30, 2023
Merged

88 support j core #108

merged 13 commits into from
Jan 30, 2023

Conversation

kevinacahalan
Copy link
Member

No description provided.

kevinacahalan and others added 11 commits January 25, 2023 15:58
* made some boxes

basic stuff

* aside

* add float to move it right

* making table

* dynamic size

* using better units

* Create tabs.rs

* screen fitting

* resolve merge conflict

* resolve merge conflict 2

* progress on regview

* changed according to clippy

* update

* reformatted - clippy
This locks the rust version to the most recent available (1.67.0). Future releases will be ignored by GitHub Actions until this version is manually updated. The rust-toolchain.toml file also forces clients to have the specific version of the toolchain for the project to avoid compatibility issues.

In addition, the GitHub Action for installing the rust toolchain has been replaced with `rustup show`. The action is no longer actively developed as of the time of commit and does not fully support reading from the rust-toolchain.toml file from testing. While the functionality of `rustup show` installing from the toolchain file is not documented well, it will at the moment perform the same action. Should a proper "install the toolchain from the rust-toolchain.toml file" feature be released, this would be preferred.

Finally, as a bonus, *all* tests in check.yml will now run regardless of how each test completed. This, for example, allows for testing the build of the project even if there was a small formatting issue reported by `cargo fmt`. Previously, a formatting issue would have stopped all checking and possibly hide other problems that could be fixed at the same time.
Need to write test cases
@kevinacahalan kevinacahalan linked an issue Jan 30, 2023 that may be closed by this pull request
Copy link
Member
@jerrettl jerrettl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍

See comment for one detail

Co-authored-by: Jerrett Longworth <jerrett7@gmail.com>
@kevinacahalan kevinacahalan merged commit 86be8d0 into main Jan 30, 2023
@kevinacahalan kevinacahalan deleted the 88-support-j-core branch January 30, 2023 21:38
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.

Support J (Core)
3 participants
0