Eddiebot: Addition of an ignoreWords.ts file

Created on 1 Dec 2020  路  7Comments  路  Source: EddieJaoudeCommunity/EddieBot

Description:
The creation of a separate class which consists of the list of words to be ignored.

Advantages:

  • Makes the process of adding words to the ignore list easier for first-timers.
  • It adds a layer of abstraction in making changes to the config.ts file.

image

All 7 comments

It's great having you contribute to this project

Feel free to raise an Issue! Welcome to the community :nerd_face:

If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 馃 . Any questions let us know.

This is a good idea(might make it easier for new contributors to find the file).
However not sure if separating this from config file is okay or not...

I feel that if we had better documentation for the Alex.js config that would solve this problem. I don't think we should maintain many small config files.

What you think?

I feel that if we had better documentation for the Alex.js config that would solve this problem. I don't think we should maintain many small config files.

What you think?

Yes, keeping one config would keep things cleaner.
Perhaps we should improve the documentation on how to add a new word to the ignore-list and about the Alex.js config

I think there is an issue for this #320

I agree it needs to be made clearer 馃憤

Especially what strings need to be included in the config for the words to be ignored, here is the link to the library that Alexjs uses https://github.com/retextjs/retext-equality/blob/main/rules.md

Was this page helpful?
0 / 5 - 0 ratings