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.
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.
Comment on lines
+4
to
+6
| 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.
issue (typo): Typo: 'intially' should be 'initially'.
Please update 'intially' to 'initially'.
Suggested change
| 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. |
Comment on lines
+8
to
+9
| 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.
issue (typo): Correct capitalization: 'PyPi' should be 'PyPI'.
Please update 'PyPi' to 'PyPI' for accuracy.
Suggested change
| 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. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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: