Skip to content
This repository has been archived by the owner on Jul 11, 2022. It is now read-only.

Integrate lezer-promql v0.19.0 with alternative top #142

Merged
merged 8 commits into from
May 20, 2021

Conversation

Nexucis
Copy link
Member

@Nexucis Nexucis commented May 19, 2021

No description provided.

@Nexucis Nexucis requested a review from juliusv May 19, 2021 13:56
Signed-off-by: Augustin Husson <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented May 19, 2021

Codecov Report

Merging #142 (b1a0cf5) into master (197c219) will decrease coverage by 0.32%.
The diff coverage is 63.63%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #142      +/-   ##
==========================================
- Coverage   72.72%   72.40%   -0.33%     
==========================================
  Files          15       15              
  Lines         748      750       +2     
  Branches      167      167              
==========================================
- Hits          544      543       -1     
- Misses        125      129       +4     
+ Partials       79       78       -1     
Impacted Files Coverage Δ
src/lang-promql/complete/hybrid.ts 74.69% <63.63%> (-1.03%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 197c219...b1a0cf5. Read the comment docs.

src/app/app.html Outdated Show resolved Hide resolved
src/lang-promql/promql.ts Outdated Show resolved Hide resolved
@Nexucis Nexucis force-pushed the feature/integrate-alternative-tops branch from e736c31 to 5804c31 Compare May 20, 2021 07:07
Signed-off-by: Augustin Husson <[email protected]>
src/app/app.html Outdated
@@ -34,6 +34,12 @@ <h3>CodeMirror Mode PromQL</h3>
<option selected value="offline">Offline</option>
<option value="prometheus">Prometheus</option>
</select>
<br>
<label for="fullLanguage">Language to complete</label>
Copy link
Member

Choose a reason for hiding this comment

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

I guess this and the line below now also shouldn't say fullLanguage / fullPromQLLanguage anymore, but something like languageType.

@juliusv
Copy link
Member

juliusv commented May 20, 2021

👍 besides comment

@Nexucis
Copy link
Member Author

Nexucis commented May 20, 2021

thanks for the review @juliusv !

@Nexucis Nexucis merged commit 87c07b5 into master May 20, 2021
@Nexucis Nexucis deleted the feature/integrate-alternative-tops branch May 20, 2021 12:51
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.

3 participants