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

chore: Rename HydroflowPlus to Hydro #1617

Merged
merged 12 commits into from
Dec 20, 2024
Merged

chore: Rename HydroflowPlus to Hydro #1617

merged 12 commits into from
Dec 20, 2024

Conversation

rohitkulshreshtha
Copy link
Contributor

No description provided.

Copy link

cloudflare-workers-and-pages bot commented Dec 20, 2024

Deploying hydroflow with  Cloudflare Pages  Cloudflare Pages

Latest commit: 808dce1
Status: ✅  Deploy successful!
Preview URL: https://19e46b97.hydroflow.pages.dev
Branch Preview URL: https://rohit-hfp-rename.hydroflow.pages.dev

View logs

Copy link
Contributor

@jhellerstein jhellerstein left a comment

Choose a reason for hiding this comment

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

Left you comments on merging with #1621 edits to README.md. Otherwise LGTM.

@@ -24,7 +24,7 @@ Hydro provides a high-level language for the majority of developers called [Hydr

## Development Setup

See the [quickstart section of the Hydroflow+ book](https://hydro.run/docs/hydroflow_plus/quickstart/) for instructions on installing Rust and getting started with the Hydroflow+ template.
See the [quickstart section of the Hydroflow+ book](https://hydro.run/docs/hydro/quickstart/) for instructions on installing Rust and getting started with the Hydroflow+ template.
Copy link
Contributor

Choose a reason for hiding this comment

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

Needs name change edits. I have a separate PR #1621 with suggested wording, but you'll still need to change the names of directories/files in the docs and links to those in the version in my PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep. I will merge #1621 last.

Thanks for putting that CR together!

docs/docs/hydroflow/quickstart/index.md Show resolved Hide resolved

import StageleftDocs from '../../../stageleft/README.md'

Under the hood, Hydroflow uses a library called Stageleft to power the `q!` macro and code generation logic. The following docs, from the Stageleft README, outline the core architecture of Stageleft.
Copy link
Member

Choose a reason for hiding this comment

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

*Hydro

docs/docs/hydroflow/quickstart/index.md Show resolved Hide resolved
@MingweiSamuel
Copy link
Member

Presumably something will break, but we will find out when we need to

@rohitkulshreshtha rohitkulshreshtha merged commit 162e49c into main Dec 20, 2024
14 checks passed
MingweiSamuel added a commit that referenced this pull request Dec 22, 2024
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