Skip to content

Conversation

clementbolin
Copy link
Contributor

What does this PR do ?

Fix bad links in the documentation

@clementbolin clementbolin requested a review from Aschen April 13, 2022 08:44
@clementbolin clementbolin self-assigned this Apr 13, 2022
@clementbolin clementbolin changed the title Fix/doc event Fix Bad links in the Kuzzle event documentation Apr 13, 2022
@clementbolin clementbolin changed the base branch from master to 7-dev April 13, 2022 08:45
@codecov
Copy link

codecov bot commented Apr 13, 2022

Codecov Report

Merging #695 (d614ead) into 7-dev (c6636e3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head d614ead differs from pull request most recent head af0db21. Consider uploading reports for the commit af0db21 to get more accurate results

@@           Coverage Diff           @@
##            7-dev     #695   +/-   ##
=======================================
  Coverage   83.47%   83.48%           
=======================================
  Files          45       45           
  Lines        2094     2095    +1     
  Branches      370      370           
=======================================
+ Hits         1748     1749    +1     
  Misses        268      268           
  Partials       78       78           
Impacted Files Coverage Δ
src/KuzzleError.ts 100.00% <100.00%> (ø)

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 c6636e3...af0db21. Read the comment docs.

@Aschen
Copy link
Contributor

Aschen commented Apr 15, 2022

@clementbolin PR fixing the documentation can be done directly on master so they will deploy the production version of the documentation when merged

@clementbolin clementbolin changed the base branch from 7-dev to master April 15, 2022 13:26
@clementbolin clementbolin merged commit a8e94a1 into master Apr 15, 2022
@clementbolin clementbolin deleted the fix/doc-event branch April 15, 2022 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants