Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

fixed regex bug on plugin installation match (related to #84) #156

Merged
merged 1 commit into from
May 27, 2022

Conversation

sio4
Copy link
Member

@sio4 sio4 commented May 20, 2022

I made a mistake when I wrote a regex for NewPlugin(). It prevented adding a version tag when running the buffalo plugin install command. This PR fixed that issue and added test cases.

Issue

  • buffalo plugin install github.com/gobuffalo/buffalo-awesome/v2 works
  • buffalo plugin install github.com/gobuffalo/buffalo-awesome/v2@latest works
  • buffalo plugin install github.com/gobuffalo/buffalo-awesome works
  • buffalo plugin install github.com/gobuffalo/buffalo-awesome@latest doesn't work
    • was interpreted as buffalo-awesome@latest is the command name

Fix

  • fixed regex to care @ too.

Additionally, fixed one deprecation warning on Plugins.Decode().

related previous pr: #84

@sio4 sio4 added the bug Something isn't working label May 20, 2022
@sio4 sio4 requested review from paganotoni and a team May 20, 2022 14:39
@sio4 sio4 self-assigned this May 20, 2022
@sio4 sio4 enabled auto-merge (rebase) May 21, 2022 01:55
Copy link
Member

@paganotoni paganotoni left a comment

Choose a reason for hiding this comment

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

Thanks @sio4

@sio4 sio4 merged commit 22664ba into development May 27, 2022
@paganotoni paganotoni deleted the fix-plugin-regex-bug branch May 27, 2022 15:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants