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.

Sign up for GitHub

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

Jump to bottom

Add 'test_author' 'make' target (and alias 'atest') #175

Closed
wants to merge 2 commits into from
Closed

Add 'test_author' 'make' target (and alias 'atest') #175

wants to merge 2 commits into from

Commits on Apr 30, 2021

  1. Add 'test_author' 'make' target (and alias 'atest') 

    Add envvar NYTPROF_AUTHOR_TESTING.  'make test_author' and 'make atest'
will set this envvar true, thereby causing certain unit tests within the
test suite and/or test files to run only when needed by NYTProf
developers.

Accordingly, move 90-pod.t back into t/ from xt/, but have this run only
when NYTPROF_AUTHOR_TESTING is set.

This should have a favorable impact on
timbunce#174.
    @jkeenan
    jkeenan committed Apr 30, 2021
    Configuration menu
    Copy the full SHA
    8e3af69 View commit details
    Browse the repository at this point in the history

  2. Remove unnecessary definitions in Makefile.PL 

    These 2 NYTPROF_* targets don't need to be defined outside the scope of
a particular 'make' target.  Defining them may interfere with, e.g.,
.appveyor.yml.
    @jkeenan
    jkeenan committed Apr 30, 2021
    Configuration menu
    Copy the full SHA
    14899c9 View commit details
    Browse the repository at this point in the history