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

PropertyDDS and lint rules #10316

Closed
tylerbutler opened this issue May 17, 2022 · 3 comments
Closed

PropertyDDS and lint rules #10316

tylerbutler opened this issue May 17, 2022 · 3 comments

Comments

@tylerbutler
Copy link
Member

As we apply more lint rule changes (see #9501), the PropertyDDS packages stand out because they have different code styles and standards, and lack of familiarity with the code makes it tough for some folks to apply changes safely. This leads to us disabling a lot of new rules in the PropertyDDS packages, which isn't ideal.

This issue exists to track a better solution, if one exists.

@DLehenbauer
Copy link
Contributor

When choosing how much to invest in linting Property DDS, keep in mind that we plan to remove it after migrating customers to '/packages/dds/tree'.

@tylerbutler
Copy link
Member Author

tylerbutler commented Jun 14, 2022

@Josmithr @skylerjokiel @ChumpChief Just FYI - see Daniel's comment. Especially relevant to the lint work we've been doing.

@ghost ghost added the status: stale label Dec 12, 2022
@ghost
Copy link

ghost commented Dec 12, 2022

This issue has been automatically marked as stale because it has had no activity for 180 days. It will be closed if no further activity occurs within 8 days of this comment. Thank you for your contributions to Fluid Framework!

@ghost ghost closed this as completed Dec 20, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants