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

fix running URL display #1206

Merged
merged 1 commit into from
Mar 2, 2021
Merged

fix running URL display #1206

merged 1 commit into from
Mar 2, 2021

Conversation

fharper
Copy link
Contributor

@fharper fharper commented Jan 26, 2021

by doing that you can just click on it in the terminal if you want

by doing that you can just click on it in the terminal if you want
@fharper fharper self-assigned this Jan 26, 2021
@fharper
Copy link
Contributor Author

fharper commented Jan 26, 2021

@MarinPostma I cannot compile MeiliSearch on my computer, so I wasn't able to test, but I assume it will work since I just updated a string...

Also, let me know if I needed to tag someone else.

@fharper fharper added the enhancement New feature or improvement label Jan 26, 2021
@codecov
Copy link

codecov bot commented Jan 26, 2021

Codecov Report

Merging #1206 (d01c93a) into master (6d79107) will increase coverage by 2.43%.
The diff coverage is 85.97%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1206      +/-   ##
==========================================
+ Coverage   77.52%   79.95%   +2.43%     
==========================================
  Files         105      105              
  Lines       12277    12154     -123     
==========================================
+ Hits         9518     9718     +200     
+ Misses       2759     2436     -323     
Impacted Files Coverage Δ
meilisearch-core/src/automaton/mod.rs 23.37% <ø> (-4.68%) ⬇️
meilisearch-http/src/analytics.rs 0.00% <0.00%> (ø)
meilisearch-http/src/helpers/authentication.rs 33.33% <0.00%> (-5.38%) ⬇️
meilisearch-http/src/main.rs 2.77% <0.00%> (ø)
meilisearch-http/src/option.rs 52.69% <ø> (-0.41%) ⬇️
meilisearch-http/tests/common.rs 90.04% <ø> (+1.40%) ⬆️
meilisearch-schema/src/position_map.rs 93.87% <ø> (ø)
meilisearch-schema/src/schema.rs 98.38% <ø> (+0.39%) ⬆️
meilisearch-core/src/database.rs 87.13% <50.00%> (+0.15%) ⬆️
meilisearch-core/src/query_tree.rs 78.31% <85.71%> (-0.11%) ⬇️
... and 29 more

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 ff38220...d01c93a. Read the comment docs.

@MarinPostma
Copy link
Contributor

Uhm would that work with tls?

@fharper
Copy link
Contributor Author

fharper commented Jan 26, 2021

You may have a warning as I changed only the display of the running URL.

Copy link
Member

@curquiza curquiza left a comment

Choose a reason for hiding this comment

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

checked with @qdequele!
bors merge

@bors
Copy link
Contributor

bors bot commented Mar 2, 2021

Build succeeded:

@bors bors bot merged commit a77d517 into master Mar 2, 2021
@bors bors bot deleted the fharper/fix-listening-url branch March 2, 2021 10:26
bors bot added a commit that referenced this pull request Mar 26, 2021
1310: Fix display of http address r=MarinPostma a=curquiza

Wrong display introduced by #1206

Now displaying:

<img width="968" alt="Capture d’écran 2021-03-26 à 12 04 59" src="https://user-images.githubusercontent.com/20380692/112622594-8c173080-8e2b-11eb-81c3-5876d273e5fa.png">


Co-authored-by: Clémentine Urquizar <clementine@meilisearch.com>
bors bot added a commit that referenced this pull request Mar 29, 2021
1310: Fix display of http address r=MarinPostma a=curquiza

Wrong display introduced by #1206

Now displaying:

<img width="968" alt="Capture d’écran 2021-03-26 à 12 04 59" src="https://user-images.githubusercontent.com/20380692/112622594-8c173080-8e2b-11eb-81c3-5876d273e5fa.png">


Co-authored-by: Clémentine Urquizar <clementine@meilisearch.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants