Skip to content

Remove unused dependency: wicked#11150

Merged
andreslucena merged 1 commit intodevelopfrom
chore/remove-wicked
Aug 29, 2023
Merged

Remove unused dependency: wicked#11150
andreslucena merged 1 commit intodevelopfrom
chore/remove-wicked

Conversation

@alecslupu
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu commented Jul 11, 2023

🎩 What? Why?

After #10727, and #10731 wicked gem is not being used anymore, therefore this PR removes the unused dependency.

📌 Related Issues

Link your PR to an issue

Testing

  1. Make sure the pipeline is green
  2. Make sure Initiative wizard still works
  3. Make sure the initiative signing wizard still works

♥️ Thank you!

@alecslupu alecslupu changed the title Chore/remove wicked Remove unused wicked dependency Jul 11, 2023
@alecslupu alecslupu changed the title Remove unused wicked dependency Remove unused dependency: wicked Jul 11, 2023
@alecslupu alecslupu added dependencies Pull requests that update a dependency file or issues that talk about updating dependencies module: initiatives labels Jul 11, 2023
@alecslupu alecslupu force-pushed the chore/remove-wicked branch 2 times, most recently from 1b09307 to 2466327 Compare July 11, 2023 10:55
@alecslupu alecslupu added the type: change PRs that implement a change for an existing feature label Jul 12, 2023
@andreslucena
Copy link
Copy Markdown
Member

@alecslupu can we retake this one? Probably starting over from develop and removing the gem from the gemspec/Gemfiles/etc would be enough, I think

@alecslupu alecslupu force-pushed the chore/remove-wicked branch from 2466327 to 42429d4 Compare August 29, 2023 11:42
@alecslupu alecslupu marked this pull request as ready for review August 29, 2023 12:51
@alecslupu alecslupu requested a review from a team August 29, 2023 12:52
@alecslupu alecslupu added this to the 0.28.0 milestone Aug 29, 2023
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

🔝

The only wicked that I found is the wicked-pdf gem, so perfect!

@andreslucena andreslucena merged commit e85930e into develop Aug 29, 2023
@andreslucena andreslucena deleted the chore/remove-wicked branch August 29, 2023 12:58
entantoencuanto added a commit that referenced this pull request Aug 30, 2023
* develop:
  Add recognition to BrowserStack in the README (#11546)
  Remove unused view hook for `:upcoming_meeting_for_card` (#11543)
  Remove unused dependency: `wicked` (#11150)
  Clean-up initiatives signature URLs and methods (#11545)
  Refactor initiative signing wizard (#10731)
  Fix Permissions screen on budgets throw errors (#11532)
  Redesign: read more literal (#11516)
  Fix 'download your data' when there are comments on budgets (#11531)
entantoencuanto added a commit that referenced this pull request Sep 8, 2023
…gn-staging

* fix/activities-block-follow-button: (27 commits)
  Add tests to follow button in processes and assemblies landing page
  Add follow button to participatory spaces last activities content block
  Remove duplication from participatory spaces publications controllers (#11549)
  Fix the a11y tool icons with redesign (#11175)
  Remove duplication from amendments events specs (#11553)
  Remove duplication from elections' user roles forms (#11548)
  Update Node.js from v16.13.0 to v18.17.1 (#11564)
  Remove duplication from stats presenters (#11551)
  Fix Bootsnap configuration (#11483)
  Remove duplication for add questions specs examples (#11559)
  Remove duplication from invites queries (#11552)
  Fix typos and copy-paste errors from comments and examples (#11536)
  Fix conference venues meetings visibility (#11542)
  Add recognition to BrowserStack in the README (#11546)
  Remove unused view hook for `:upcoming_meeting_for_card` (#11543)
  Remove unused dependency: `wicked` (#11150)
  Clean-up initiatives signature URLs and methods (#11545)
  Refactor initiative signing wizard (#10731)
  Fix Permissions screen on budgets throw errors (#11532)
  Redesign: read more literal (#11516)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file or issues that talk about updating dependencies module: initiatives target:cleanup type: change PRs that implement a change for an existing feature

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants