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

Adding 378 lists #628

Merged
merged 18 commits into from
Nov 26, 2018
Merged

Conversation

DandelionSprout
Copy link
Contributor

Holy cow, did I hit a goldmine while researching another repo. 🎉

@codecov
Copy link

codecov bot commented Nov 21, 2018

Codecov Report

Merging #628 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #628   +/-   ##
======================================
  Coverage    89.9%   89.9%           
======================================
  Files         185     185           
  Lines       24757   24757           
======================================
  Hits        22259   22259           
  Misses       2498    2498

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 58972d4...5185bcf. Read the comment docs.

Copy link
Contributor

@bogachenko bogachenko left a comment

Choose a reason for hiding this comment

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

@DandelionSprout in this committee (data/FilterList.json), travis complains about line 13096 there is no date in the "publishedDate" section

upd = it is written there "publishedDate": "(Domains)" -> In the place of "Domains", the date is needed?

no?

@DandelionSprout
Copy link
Contributor Author

Ah, so that was why the build was failing. Thanks for spotting it. 👍

@bogachenko
Copy link
Contributor

bogachenko commented Nov 21, 2018

you forgot to delete again data/FilterListMaintainer.json -> 1188

Copy link
Contributor

@bogachenko bogachenko left a comment

Choose a reason for hiding this comment

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

"filterListId": 1188,

@DandelionSprout
Copy link
Contributor Author

The list with ID 1188 was a phantom resulting from miscopying during my info gathering process, so there's no reason to keep that "list" around. So I removed it in its entirety.

@bogachenko
Copy link
Contributor

Yes. I'm catching on. but the remnants of from this list of, too, need to remove

@DandelionSprout
Copy link
Contributor Author

Ah, okay. I hope I've done so now.

@bogachenko
Copy link
Contributor

:shipit: ez

@DandelionSprout
Copy link
Contributor Author

I've now also made some small changes in various files, reflecting that my main Norwegian list is now a Nordic list.

It's got 30-ish Malaysian-relevant entries, that I'm aware of.
@bogachenko
Copy link
Contributor

bogachenko commented Nov 25, 2018

I think @collinbarrett is dead 😞
he has not been with us for 8 days
default
Why God? Why?

@collinbarrett
Copy link
Owner

lol. not dead. on vacation for U.S. Thanksgiving holiday and went computer-free. back now, but might take me another day to get caught up. thanks for your patience.

@bogachenko
Copy link
Contributor

bogachenko commented Nov 25, 2018

@collinbarrett hell, right (recently, black Friday was), you're having (americans) thanksgiving now. my fool

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
directory-data changes to basic FilterLists data
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants