FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Remove plugs by mgiacomini · Pull Request #69 · ateliware/triplex · GitHub

/ triplex Public

Remove plugs - #69

Open
mgiacomini wants to merge 3 commits into
masterfrom
remove-plugs
ConversationCommits (3)ChecksFiles changed
Open

Remove plugs#69
mgiacomini wants to merge 3 commits into
masterfrom
remove-plugs

Conversation

mgiacomini commented Jun 8, 2019

Copy link
Copy Markdown
Contributor

Following the #63, this pull request remove all plugs in favor of a new lib.

coveralls commented Jun 8, 2019 •
edited
Loading

Copy link
Copy Markdown

Coverage decreased (-0.08%) to 99.422% when pulling ccd6bf8 on remove-plugs into bdf19bb on master.

mgiacomini commented Jul 29, 2019

Copy link
Copy Markdown
Contributor Author

https://github.com/ateliware/triplex_plug

kelvinst suggested changes Sep 16, 2019
View reviewed changes

kelvinst left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Just remember to remove the references to plugs on README. Maybe mentioning that that now they're on triplex_plugs lib.

kelvinst commented Sep 16, 2019

Copy link
Copy Markdown
Contributor

Also, once we merge #60, we must remove that one from here and move it there.

mgiacomini commented Oct 23, 2019

Copy link
Copy Markdown
Contributor Author

Just remember to remove the references to plugs on README. Maybe mentioning that that now they're on triplex_plugs lib.

yes! I will work to improve the docs too.

kelvinst commented Oct 20, 2020

Copy link
Copy Markdown
Contributor

@mgiacomini were you able to work on it? If not, can we send the task to someone else? I can take over if you want.

kelvinst added this to the 2.0 milestone May 29, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
kelvinst kelvinst requested changes
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2.0

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL