-
Notifications
You must be signed in to change notification settings - Fork 1
change note #3
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
change note #3
Conversation
Reviewer's GuideThis PR revises the README’s introductory note to reflect that the project has transitioned from a fork to a standalone repository and updates the release status details. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @melonora - I've reviewed your changes - here's some feedback:
- Fix the typo 'intially' in the README to 'initially'.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Fix the typo 'intially' in the README to 'initially'.
## Individual Comments
### Comment 1
<location> `README.md:4` </location>
<code_context>
work of the developer on this repo. That being said, sadly the state on main has been the case that the code was
-deleted and there has been no development for a while. Therefore, we have decided to fork the repository and
-continue development here, where the community is better able to contribute to and maintain the project.
+deleted and there has been no development for a while. Therefore, we intially decided to fork the repository and
+continue development here, where the community is better able to contribute to and maintain the project. We now changed
+it into a standalone repository.
</code_context>
<issue_to_address>
Typo: 'intially' should be 'initially'.
Please update 'intially' to 'initially'.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
deleted and there has been no development for a while. Therefore, we intially decided to fork the repository and
continue development here, where the community is better able to contribute to and maintain the project. We now changed
it into a standalone repository.
=======
deleted and there has been no development for a while. Therefore, we initially decided to fork the repository and
continue development here, where the community is better able to contribute to and maintain the project. We now changed
it into a standalone repository.
>>>>>>> REPLACE
</suggested_fix>
### Comment 2
<location> `README.md:8` </location>
<code_context>
+it into a standalone repository.
-Note, the fork currently is not released. We are trying to get the name `xarray-dataclasses` on PyPi.
+Note, the repository is not yet released. We are trying to get the name `xarray-dataclasses` on PyPi and make some
+additional changes before release.
</code_context>
<issue_to_address>
Correct capitalization: 'PyPi' should be 'PyPI'.
Please update 'PyPi' to 'PyPI' for accuracy.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
Note, the repository is not yet released. We are trying to get the name `xarray-dataclasses` on PyPi and make some
additional changes before release.
=======
Note, the repository is not yet released. We are trying to get the name `xarray-dataclasses` on PyPI and make some
additional changes before release.
>>>>>>> REPLACE
</suggested_fix>
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
deleted and there has been no development for a while. Therefore, we intially decided to fork the repository and | ||
continue development here, where the community is better able to contribute to and maintain the project. We now changed | ||
it into a standalone repository. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
issue (typo): Typo: 'intially' should be 'initially'.
Please update 'intially' to 'initially'.
deleted and there has been no development for a while. Therefore, we intially decided to fork the repository and | |
continue development here, where the community is better able to contribute to and maintain the project. We now changed | |
it into a standalone repository. | |
deleted and there has been no development for a while. Therefore, we initially decided to fork the repository and | |
continue development here, where the community is better able to contribute to and maintain the project. We now changed | |
it into a standalone repository. |
Note, the repository is not yet released. We are trying to get the name `xarray-dataclasses` on PyPi and make some | ||
additional changes before release. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
issue (typo): Correct capitalization: 'PyPi' should be 'PyPI'.
Please update 'PyPi' to 'PyPI' for accuracy.
Note, the repository is not yet released. We are trying to get the name `xarray-dataclasses` on PyPi and make some | |
additional changes before release. | |
Note, the repository is not yet released. We are trying to get the name `xarray-dataclasses` on PyPI and make some | |
additional changes before release. |
Change note in preparation for making the repo standalone.
Summary by Sourcery
Update the project README to reflect that the repository has transitioned from a fork to a standalone project and clarify its release status on PyPI.
Documentation: