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

Remove dupe and #2772

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

Remove dupe and #2772

wants to merge 6 commits into from

Conversation

cuihtlauac
Copy link
Collaborator

@cuihtlauac cuihtlauac commented Oct 14, 2024

Fix #2771

Note sure we should fix stuff from data/planet

README.md Outdated
Comment on lines 22 to 23
We generate OCaml code from this data to serve the site content and OCurrent to
automate the data pipelines. All the data used in the site can be found in [`./data`](./data).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
We generate OCaml code from this data to serve the site content and OCurrent to
automate the data pipelines. All the data used in the site can be found in [`./data`](./data).
We generate OCaml code from this data to serve the site content and for OCurrent to
automate the data pipelines. All the data used in the site can be found in [`./data`](./data).

Does "we generate OCaml code" refer to both "serve the site content" and "OCurrent to automate" ?? Or is OCurrent automating separate from this code generation?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I've rewritten this sentence

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.

Double wording on front page
2 participants