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

Remove feature flags within the scope of pkg #7511

Merged
merged 14 commits into from
Oct 14, 2020
Merged

Conversation

terencechain
Copy link
Member

Part of #7509

This PR removed feature flags that were in scope of pkg. It means these feature flags weren't used which makes it easier to reason for removing

@terencechain terencechain added Ready For Review A pull request ready for code review OK to merge labels Oct 12, 2020
@terencechain terencechain requested a review from a team as a code owner October 12, 2020 18:28
@terencechain terencechain self-assigned this Oct 12, 2020
@codecov
Copy link

codecov bot commented Oct 12, 2020

Codecov Report

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

@@           Coverage Diff           @@
##           master    #7511   +/-   ##
=======================================
  Coverage   60.66%   60.66%           
=======================================
  Files         431      431           
  Lines       30624    30624           
=======================================
  Hits        18577    18577           
  Misses       9020     9020           
  Partials     3027     3027           

prestonvanloon
prestonvanloon previously approved these changes Oct 14, 2020
@prestonvanloon prestonvanloon added this to the v1.0.0-beta milestone Oct 14, 2020
@prylabs-bulldozer prylabs-bulldozer bot merged commit e5ed2cd into master Oct 14, 2020
@delete-merged-branch delete-merged-branch bot deleted the ff-cleanup-part1 branch October 14, 2020 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review A pull request ready for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants