Skip to content

chore(celest): Update linter/style preferences#404

Merged
dnys1 merged 1 commit intomainfrom
05-09-fix_celest_analyzer_plugin_cannot_be_part_of_workspace
May 9, 2025
Merged

chore(celest): Update linter/style preferences#404
dnys1 merged 1 commit intomainfrom
05-09-fix_celest_analyzer_plugin_cannot_be_part_of_workspace

Conversation

@dnys1
Copy link
Member

@dnys1 dnys1 commented May 9, 2025

  • Ensures celest_lints is used
  • Applies latest linter/style preferences using dart fix
  • Re-formats using Dart 3.8

Includes fix for analyzer plugin, which cannot be part of the workspace. Since the plugin's source code is copied to a temp directory, it will be outside the workspace which causes errors activating it.

@dnys1 dnys1 force-pushed the 05-09-fix_celest_analyzer_plugin_cannot_be_part_of_workspace branch from ccdd044 to d1bcf97 Compare May 9, 2025 17:50
@dnys1 dnys1 changed the title fix(celest): Analyzer plugin cannot be part of workspace chore(celest): Update linter/style preferences May 9, 2025
@dnys1 dnys1 force-pushed the 05-09-fix_celest_analyzer_plugin_cannot_be_part_of_workspace branch from d1bcf97 to 46761dd Compare May 9, 2025 17:54
Copy link
Member Author

dnys1 commented May 9, 2025

Merge activity

  • May 9, 11:22 AM PDT: A user started a stack merge that includes this pull request via Graphite.
  • May 9, 11:25 AM PDT: Graphite rebased this pull request as part of a merge.
  • May 9, 11:25 AM PDT: @dnys1 merged this pull request with Graphite.

@dnys1 dnys1 changed the base branch from 05-09-feat_lints_update_linter_style_preferences to graphite-base/404 May 9, 2025 18:23
@dnys1 dnys1 changed the base branch from graphite-base/404 to main May 9, 2025 18:23
- Ensures `celest_lints` is used
- Applies latest linter/style preferences using dart fix
- Re-formats using Dart 3.8

Includes fix for analyzer plugin, which cannot be part of the workspace. Since the plugin's source code is copied to a temp directory, it will be outside the workspace which causes errors activating it.
@dnys1 dnys1 force-pushed the 05-09-fix_celest_analyzer_plugin_cannot_be_part_of_workspace branch from 46761dd to 0e4ebe2 Compare May 9, 2025 18:24
@dnys1 dnys1 merged commit 39e15d9 into main May 9, 2025
5 checks passed
@dnys1 dnys1 deleted the 05-09-fix_celest_analyzer_plugin_cannot_be_part_of_workspace branch May 9, 2025 18:25
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.

1 participant