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

refactor: only export custom prop-type functions #254

Merged
merged 13 commits into from
Sep 19, 2021

Conversation

HendrikThePendric
Copy link
Contributor

No description provided.

BREAKING CHANGE: we no longer append to and re-export the `PropTypes`
object from the `prop-types` package. Instead it will need to be installed
by the consumer and regular prop-types need to be imported from there. Only
the DHIS2 custom prop-types can be imported from this package from now on.
BREAKING CHANGE: regular prop-types can no longer be imported from @dhis2/prop-types
BREAKING CHANGE: DHIS2 custom prop-types are only exported as named functions
@HendrikThePendric HendrikThePendric changed the title CLI-218-only-export-own-prop-types-functions refactor: only export custom prop-type functions Sep 14, 2021
@HendrikThePendric HendrikThePendric marked this pull request as draft September 15, 2021 06:36
@HendrikThePendric
Copy link
Contributor Author

Converted this PR to draft as I uncovered some prop-types that are not transformed correctly yet. Specifically function arguments, and arrays (often also used in function arguments) are not supported yet, but they need to be.

@HendrikThePendric HendrikThePendric self-assigned this Sep 16, 2021
@HendrikThePendric HendrikThePendric marked this pull request as ready for review September 16, 2021 10:47
@HendrikThePendric HendrikThePendric changed the base branch from master to beta September 16, 2021 10:53
Copy link
Contributor

@Mohammer5 Mohammer5 left a comment

Choose a reason for hiding this comment

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

LGTM

@HendrikThePendric HendrikThePendric merged commit c2262e0 into beta Sep 19, 2021
@HendrikThePendric HendrikThePendric deleted the CLI-218-only-export-own-prop-types-functions branch September 19, 2021 12:33
dhis2-bot added a commit that referenced this pull request Sep 19, 2021
# [3.0.0-beta.1](v2.0.4...v3.0.0-beta.1) (2021-09-19)

### Code Refactoring

* only export custom prop-type functions ([#254](#254)) ([c2262e0](c2262e0))

### BREAKING CHANGES

* prop-types from the `props-types` package are no longer re-exported from `@dhis2/prop-types`
* `@dhis2/prop-types` no longer exports a default, the DHIS2 custom prop-types are only available as named imports
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 3.0.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

dhis2-bot added a commit that referenced this pull request Sep 30, 2021
# [3.0.0](v2.0.4...v3.0.0) (2021-09-30)

### Bug Fixes

* publish codemod and add migration docs ([#260](#260)) ([7a0fc61](7a0fc61))

### Code Refactoring

* only export custom prop-type functions ([#254](#254)) ([c2262e0](c2262e0))

### BREAKING CHANGES

* prop-types from the `props-types` package are no longer re-exported from `@dhis2/prop-types`
* `@dhis2/prop-types` no longer exports a default, the DHIS2 custom prop-types are only available as named imports
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 3.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants