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

expose a public API methods from this extension #130

Merged
merged 2 commits into from
Feb 12, 2025

Conversation

lonhutt
Copy link
Contributor

@lonhutt lonhutt commented Dec 6, 2024

  • this plugin now returns an ExtensionAPI object with (currently) a single method that parses the project .bazelproject and returns a BazelProjectView object.
  • this is to allow other dependent extensions the ability to get the current state of the bazel project view without having to duplicate our parsing code
  • Added a test to validate the public methods returns some value

Copy link

github-actions bot commented Dec 6, 2024

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 102f00e.

♻️ This comment has been updated with latest results.

@lonhutt lonhutt force-pushed the add_extension_public_api branch from 3aad86a to d95180d Compare December 6, 2024 19:35
@lonhutt lonhutt requested a review from guw December 6, 2024 19:36
@lonhutt lonhutt force-pushed the add_extension_public_api branch from d95180d to 6ad9d29 Compare February 12, 2025 22:35
@lonhutt lonhutt requested a review from AdwaitChavan February 12, 2025 23:09
@lonhutt lonhutt merged commit a73ed6f into main Feb 12, 2025
7 checks passed
@lonhutt lonhutt deleted the add_extension_public_api branch February 12, 2025 23:41
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.

3 participants