Skip to content
This repository was archived by the owner on Mar 26, 2023. It is now read-only.

added forums keyword#56

Open
christopherpickering wants to merge 3 commits intoMarkIannucci:mainfrom
christopherpickering:patch-1
Open

added forums keyword#56
christopherpickering wants to merge 3 commits intoMarkIannucci:mainfrom
christopherpickering:patch-1

Conversation

@christopherpickering
Copy link
Copy Markdown
Contributor

for #55 , realized you already had forums called "topics", so just added the "forums" and "fm" keywords to the list :)

for MarkIannucci#55 , realized you already had forums called "topics", so just added the "forums" and "fm" keywords to the list :)
@MarkIannucci
Copy link
Copy Markdown
Owner

Merging #57 before this one created a merge conflict. I'll correct, merge and get the action to deploy the released extension soonish.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 28, 2022

Codecov Report

Merging #56 (c67ad63) into main (fc8a332) will increase coverage by 0.23%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #56      +/-   ##
==========================================
+ Coverage   90.24%   90.47%   +0.23%     
==========================================
  Files           1        1              
  Lines          41       42       +1     
==========================================
+ Hits           37       38       +1     
  Misses          4        4              
Impacted Files Coverage Δ
Extension/functions.js 90.47% <100.00%> (+0.23%) ⬆️

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 fc8a332...c67ad63. Read the comment docs.

@MarkIannucci
Copy link
Copy Markdown
Owner

@christopherpickering , I'd like to complete #54 before completing this which I think will get the build green and that should update the released version on the chrome web store getting this functionality live.

@christopherpickering
Copy link
Copy Markdown
Contributor Author

Sounds good!

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