Skip to content
This repository has been archived by the owner on Aug 5, 2024. It is now read-only.

[fix] parsing of project view file fails if the file doesn't exist | #BAZEL-21 Fixed | #215

Merged
merged 9 commits into from
Apr 11, 2022

Conversation

abrams27
Copy link
Member

@abrams27 abrams27 commented Apr 7, 2022

Now parsing of project view fails if the file doesn't exist, also now implementation is in kotlin (it still needs some love, to become a good kotlin code, but I'll do it later anyway) + mock parser is better - it uses entire logic of the real implementation


https://youtrack.jetbrains.com/issue/BAZEL-21


testing:

bazel test //executioncontext/projectview/...

@abrams27 abrams27 changed the title [fix] | #BAZEL-21 Fixed | [fix] parsing of project view file fails if file doesn't exist | #BAZEL-21 Fixed | Apr 8, 2022
@abrams27 abrams27 changed the title [fix] parsing of project view file fails if file doesn't exist | #BAZEL-21 Fixed | [fix] parsing of project view file fails if the file doesn't exist | #BAZEL-21 Fixed | Apr 8, 2022
@abrams27 abrams27 force-pushed the issue-21-project-view-parsing-fail branch from 1d9611c to b7881ec Compare April 8, 2022 08:14
@abrams27 abrams27 requested a review from lukaszwawrzyk April 8, 2022 08:18
@abrams27 abrams27 marked this pull request as ready for review April 8, 2022 08:18
@abrams27 abrams27 merged commit 56dc335 into master Apr 11, 2022
@abrams27 abrams27 deleted the issue-21-project-view-parsing-fail branch April 11, 2022 08:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants