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

DEVLIB-1257: Rename Compute@Edge to Compute #328

Merged
merged 3 commits into from
Oct 31, 2023
Merged

Conversation

doramatadora
Copy link
Contributor

@doramatadora doramatadora commented Oct 26, 2023

JIRA ticket: DEVLIB-1257

Please review and notify DevRel (and @Integralist) of the new Viceroy release 🙌🏼

@doramatadora doramatadora requested review from a team, kailan, ulyssa and kpfleming and removed request for a team and kailan October 26, 2023 17:10
kpfleming
kpfleming previously approved these changes Oct 26, 2023
Copy link
Contributor

@kpfleming kpfleming left a comment

Choose a reason for hiding this comment

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

Once the style guide has been published we'll have to revisit some of this, but it's not worth changing right now.

README.md Outdated Show resolved Hide resolved
cli/Cargo.toml Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@@ -1,6 +1,6 @@
# 🔗 compute-at-edge-abi
Copy link
Contributor

@JakeChampion JakeChampion Oct 27, 2023

Choose a reason for hiding this comment

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

@doramatadora and @kpfleming - do we want to rename this and the folder?

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh... good question. That's going to be a breaking change for anything which consumes the content here?

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe not.. it appears that the WITX files are incorporated into the crate but the directory name is not part of the crate.

Copy link
Contributor

Choose a reason for hiding this comment

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

But one of the WITX files is named compute-at-edge.witx.

@doramatadora doramatadora merged commit ab94de0 into main Oct 31, 2023
7 checks passed
@doramatadora doramatadora deleted the dora-compute-rename branch October 31, 2023 09:10
cmckendry pushed a commit to 1stdibs/Viceroy that referenced this pull request Feb 8, 2024
* DEVLIB-1257: Rename Compute@Edge to Compute

* Update README.md
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.

4 participants