Skip to content

Conversation

@evgeniy-skt
Copy link
Contributor

@evgeniy-skt evgeniy-skt commented Aug 1, 2022

Context

Can't use Layer, AllureId fields with SpecFlow Plugin

Checklist

@CLAassistant
Copy link

CLAassistant commented Aug 1, 2022

CLA assistant check
All committers have signed the CLA.

"owner": "^owner:?(.+)",
"severity": "^(normal|blocker|critical|minor|trivial)"
"severity": "^(normal|blocker|critical|minor|trivial)",
"label": "(.+)\\((.+)\\)"
Copy link
Contributor

Choose a reason for hiding this comment

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

To use same semantics as other labels better to make regex like this: "^label:([\w]+):(.+)"

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, you can use internal property in PluginConfiguration this way:

internal string label => "^label:([\\w]+):(.+)";

Copy link
Contributor

Choose a reason for hiding this comment

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

Fixed regex.

Copy link
Contributor

Choose a reason for hiding this comment

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

And I'm not sure about

internal string label => "^label:([\w]+):(.+)";

Do we really need this? In this case we can't customize a regex pattern in the config file.

};
}

public static Label CustomLabel(string key, string value)
Copy link
Contributor

Choose a reason for hiding this comment

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

There is no need to add new method, you can simply use new Label { name = key, value = value } in SpecFlow adapter directly

Copy link
Contributor

Choose a reason for hiding this comment

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

fixed

@neparij
Copy link
Contributor

neparij commented Aug 3, 2022

Hello guys! We are very glad to see your contributions.

Sounds funny because I pushed almost the same changes recently (f5ac293) :)

Your PR looks really nice, just need to make some changes on review.
I will be glad to see you in the list of contributors!

🍕 ❤️

@neparij neparij merged commit 62e9ff1 into allure-framework:main Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants