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.

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

Initial changes for the handoff from Embark to the community. #1

Merged
merged 7 commits into from
Jul 18, 2024

Conversation

LegNeato
Copy link
Collaborator

No description provided.

@LegNeato LegNeato requested a review from eddyb July 11, 2024 18:30
@LegNeato
Copy link
Collaborator Author

Note that the docs/pages links won't work until this repo is made public.

@LegNeato
Copy link
Collaborator Author

@eddyb not sure if you see GitHub review request emails

@eddyb
Copy link
Collaborator

eddyb commented Jul 13, 2024

@eddyb not sure if you see GitHub review request emails

Review requests trigger GH Mobile notifs so I do get them, I only put this off because I was looking into one specific SPIR-T bug fix but that's taking longer anyway so I should give this a once-over at least.

Copy link
Collaborator

@eddyb eddyb left a comment

Choose a reason for hiding this comment

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

Most of the comments don't require action, mainly wanted to jot my thoughts down.

.github/ISSUE_TEMPLATE/mcp.md Outdated Show resolved Hide resolved
.github/CODEOWNERS Outdated Show resolved Hide resolved
.github/workflows/ci.yaml Show resolved Hide resolved
CODE_OF_CONDUCT.md Show resolved Hide resolved
# Embark Contributor Guidelines
# Contributor Guidelines
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same here as for the code of conduct (as in it looks reasonable but no idea what the delta is to other projects).

README.md Outdated Show resolved Hide resolved
- [Cargo](https://github.com/rust-lang/cargo/) and [crates.io](https://crates.io) support to develop and publish SPIR-V crates
- High-level render graph to take advantage of this, make it easy for users to develop and use rendering effects.

## Process

We use this repo as a monorepo for everything related to the project: crates, tools, shaders, examples, tests, and design documents. This way, we can use issues and PRs covering everything in the same place, cross-reference stuff within the repo, as well as with other GitHub repos such as [rspirv](https://github.com/gfx-rs/rspirv) and [Rust](https://github.com/rust-lang/rust) itself.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh this should've been updated when SPIR-T was added, heh.

README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@eddyb
Copy link
Collaborator

eddyb commented Jul 13, 2024

$ rg -oi '(\S+|.* )embark(\S+| .*)' | rg -v '\.io/cargo-deny|(rust-ecosystem|rust-
gpu|spirt)/(pull|issues)/|Embark standard lints v0\.4'
Cargo.toml:"Embark <[email protected]>"]
LICENSE-MIT:Copyright (c) 2019-2024 Embark Studios
README.md:[Embark](https://www.embark-studios.com/).
README.md:- 2024 Embark transitions maintenance of `rust-gpu` to the open source community.
docs/src/building-rust-gpu.md:https://github.com/EmbarkStudios/rust-gpu
docs/src/building-rust-gpu.md:[SPIRV-Tools](https://github.com/EmbarkStudios/SPIRV-Tools).
docs/src/building-rust-gpu.md:https://github.com/EmbarkStudios/rust-gpu/tree/main/examples
docs/src/building-rust-gpu.md:https://github.com/EmbarkStudios/rust-gpu/tree/main/examples/runners
docs/src/building-rust-gpu.md:https://github.com/EmbarkStudios/rust-gpu/tree/main/examples/runners/ash
docs/src/building-rust-gpu.md:https://github.com/EmbarkStudios/rust-gpu/tree/main/examples/runners/cpu
docs/src/building-rust-gpu.md:https://github.com/EmbarkStudios/rust-gpu/tree/main/examples/runners/wgpu
docs/src/building-rust-gpu.md:https://github.com/EmbarkStudios/rust-gpu/tree/main/examples/shaders
crates/rustc_codegen_spirv/src/lib.rs:    clippy::match_on_vec_items, // rustc_codegen_spirv has less strict panic requirements than other embark projects
crates/spirv-std/README.md:https://github.com/EmbarkStudios/rust-gpu/blob/097ba40bedd74eeaa296e719ef7e41f2d3d76c23/docs/src/migration-to-register-tool.md
crates/spirv-std/README.md:https://github.com/EmbarkStudios/rust-gpu/blob/69cb69d28f1e64420ee31ade5e7dffb7c5621e89/examples/shaders/sky-shader/src/lib.rs

Did a quick pass, seems like there's a few more links that can be fixed maybe?
(Cargo.toml/LICENSE-MIT/README.md are false positives AFAICT)

Also this reminds me we should review Embark's set of clippy lints and figure out what's useful/overly-opinionated, and also what lints we're missing (Embark itself has updated its set of lints several times since the copies in Rust-GPU, too).

@eddyb eddyb closed this Jul 13, 2024
@eddyb
Copy link
Collaborator

eddyb commented Jul 13, 2024

Oops, wrong button, sorry!

@eddyb eddyb reopened this Jul 13, 2024
* @eddyb

/examples/ @fu5ha @VZout
* @eddyb @LegNeato @fornwall
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Note I added him to the repo but this will show an error in the UI until he accepts.

@@ -1,5 +1,3 @@
Copyright (c) 2019 Embark Studios

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Note we are removing copyright entirely, similar to the rust project's MIT license file.

@LegNeato
Copy link
Collaborator Author

@eddyb We good here? I want to get this moving so we can announce this week.

@LegNeato LegNeato merged commit e54dd75 into main Jul 18, 2024
5 of 7 checks passed
@LegNeato LegNeato deleted the embark_handoff branch July 18, 2024 15:21
@nyabinary
Copy link

nyabinary commented Aug 17, 2024

Is there a Matrix or Discord for this or anything else? :P

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.

3 participants