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

Major updates for usability #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Neo-Desktop
Copy link

Hi there - I did a bit of an overhaul of the code.

I intended to use this in my Discord Bot, but I couldn't import the generator without it having a copy of emojis.json in the current directory during runtime.

I initially made changes to compile in the database to avoid the runtime requirement.

These changes do break compatibility (I eliminated the config structure during my changes) so please use/merge this at your discretion.

If you do decide to merge this (and after any other changes are made) please feel free to add a tag noting a release version (i.e. v1.0.0) for go module versioning purposes

I will be maintaining my fork of this branch over at NeoDesktop/emojipasta

Thanks for the beautiful groundwork - I figured it's only fair to contribute back to the source repo :)


Changelog:

Use edb.prebuilt.json from emojibot2
Provide example usage
Update readme to reflect changes
Simplify code/directory structure

new file:   .gitignore
modified:   Emojipasta.go
modified:   README.md
deleted:    assets/emojis.json
new file:   edb.prebuilt.json
new file:   example/main.go
deleted:    generator/generator.go

Use edb.prebuilt.json from emojibot2
Provide example usage
Update readme to reflect chanegs

Simplify code/directory structure

	new file:   .gitignore
	modified:   Emojipasta.go
	modified:   README.md
	deleted:    assets/emojis.json
	new file:   edb.prebuilt.json
	new file:   example/main.go
	deleted:    generator/generator.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant