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

generate_bear_requirements.py: Add Perl Meta #2896

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ishaan29
Copy link

This enhances generate_bear_requirements.py to support
perl bears and generate Makefile.PL using jinja2
template.
closes #2483

This enhances generate_bear_requirements.py to support
perl bears and generate Makefile.PL using jinja2
template.
closes coala#2483
@gitmate-bot
Copy link
Collaborator

Comment on 7fcd7f5.

No newline found between shortlog and body at HEAD commit. Please add one.

Origin: GitCommitBear, Section: all.commit.

@KVGarg
Copy link
Contributor

KVGarg commented Apr 11, 2019

@Ishaan29 There should be a blank line between Closes And your commit description 😇

@@ -1,6 +1,10 @@
from coalib.bearlib.abstractions.Linter import linter
from dependency_management.requirements.DistributionRequirement import (
DistributionRequirement)
from dependency_management.requirements.AnyOneOfRequirements import (
AnyOneOfRequirements)
from dependency_managment.requirements.PerlRequirement import (
Copy link
Member

Choose a reason for hiding this comment

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

Is this dependency manager available in the coala package_manager?

Copy link
Author

Choose a reason for hiding this comment

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

No it is not available, Thank you for pointing me in the right direction ... I will create PerlRequirement file.

Copy link
Member

@sangamcse sangamcse Apr 14, 2019

Choose a reason for hiding this comment

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

BTW the issue is not about creating a dependency. This issue is to create just Perl meta.
NM.

Copy link
Author

Choose a reason for hiding this comment

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

I don't think there is a need to add PerlRequirement as CpanRequirement is already present..

Copy link
Member

Choose a reason for hiding this comment

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

If CpanRequirement is the requirement here, then work with that. Adding PerlRequirement here without having it in package_manager will break it

Copy link
Member

Choose a reason for hiding this comment

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

Will you be fixing this @Ishaan29 ?

@jayvdb
Copy link
Member

jayvdb commented May 1, 2019

Continuation of #2888

WriteMakefile(
NAME => 'Coala::Bears',
VERSION => '0.10',
PREREQ_PM => {Perl::Critic => 1.126},
Copy link
Member

Choose a reason for hiding this comment

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

this isnt a template if it has literal values in it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Generate Perl package metadata
5 participants