Skip to content

Remove IPolicyPlugin#971

Merged
shargon merged 1 commit intomasterfrom
3.0/remove-policy-plugin
Jul 31, 2019
Merged

Remove IPolicyPlugin#971
shargon merged 1 commit intomasterfrom
3.0/remove-policy-plugin

Conversation

@erikzhang
Copy link
Copy Markdown
Member

@erikzhang erikzhang commented Jul 31, 2019

Closes #963

@erikzhang erikzhang added this to the NEO 3.0 milestone Jul 31, 2019
Copy link
Copy Markdown
Member

@vncoelho vncoelho left a comment

Choose a reason for hiding this comment

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

Bye bye, good friend. 💃
It has been good to see this as plugin, @erikzhang.

However, the Native contacts in outstanding in beauty and flexibility!

@vncoelho
Copy link
Copy Markdown
Member

closes #963

@vncoelho
Copy link
Copy Markdown
Member

closes neo-project/proposals#80

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jul 31, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@cd99489). Click here to learn what that means.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #971   +/-   ##
=========================================
  Coverage          ?   53.77%           
=========================================
  Files             ?      194           
  Lines             ?    13250           
  Branches          ?        0           
=========================================
  Hits              ?     7125           
  Misses            ?     6125           
  Partials          ?        0
Impacted Files Coverage Δ
neo/Plugins/Plugin.cs 18.96% <ø> (ø)
neo/Consensus/ConsensusContext.cs 54.77% <0%> (ø)
neo/Consensus/ConsensusService.cs 15.7% <0%> (ø)
neo/Ledger/Blockchain.cs 37.5% <0%> (ø)
neo/SmartContract/Native/PolicyContract.cs 93.02% <0%> (ø)

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 cd99489...4df699d. Read the comment docs.

Copy link
Copy Markdown
Contributor

@igormcoelho igormcoelho left a comment

Choose a reason for hiding this comment

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

Very good, native policy now.

@shargon shargon merged commit 46ed67f into master Jul 31, 2019
@shargon shargon deleted the 3.0/remove-policy-plugin branch July 31, 2019 12:29
@erikzhang erikzhang removed this from the NEO 3.0 milestone Dec 6, 2019
Tommo-L pushed a commit to Tommo-L/neo that referenced this pull request Jun 22, 2020
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.

Remove FilterForBlock & FilterForMemoryPool from plugins

5 participants