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

Automate maintainer-approval flag & Improve cat/port detection #4

Open
wants to merge 1 commit into
base: freebsd-local
Choose a base branch
from

Conversation

mmokhi
Copy link
Contributor

@mmokhi mmokhi commented Dec 31, 2015

modified: extensions/BFBSD/lib/Helpers.pm

  • added some constant values
    • from AA module to here for using in other modules [like FBSDAttachments:D]
    • a value for listing all categories (in order to be used in regexes)
  • editted regexes to find "category/port" string in better way

new file: extensions/FBSDAttachments/Config.pm

  • Add config file for "FBSDAttachments" module

new file: extensions/FBSDAttachments/Extension.pm

  • FBSDAttachments automation module itself

modified: extensions/FBSDAutoAssign/Extension.pm

  • Improve regular expression to improve category/port detection (and fixing reported bugs)
  • if we assume we know all category names we can change regex in a way to search for strings in form of (category name from our valid pool)/name of port in way of regex :D
    port names are strings in type of words(alphanumeric and _) and + and - and . but we know end of port name isn't '.' :)

This PR addresses:

	modified:   extensions/BFBSD/lib/Helpers.pm 		==> 1) added some constant values 1-a)from AA module to here for using in other modules [like FBSDAttachments:D] 1-b)a value for listing all categories (in order to be used in regexes)  2) editted regexes to find "category/port" string in better way*
	new file:   extensions/FBSDAttachments/Config.pm	==> config file for "FBSDAttachments" module
	new file:   extensions/FBSDAttachments/Extension.pm	==> FBSDAttachments module itself :D
	modified:   extensions/FBSDAutoAssign/Extension.pm	==> changed regexes to find "category/port" string in better way*
* if we assume we know all category names we can change regex in a way to search for strings in form of (category name from our valid pool)/name of port in way of regex :D
@koobs koobs changed the title Changes to be committed: Automate maintainer-approval flag & Improve cat/port detection Dec 31, 2015
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.

1 participant