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

test: define default clang args for testing #256

Closed
wants to merge 1 commit into from
Closed

test: define default clang args for testing #256

wants to merge 1 commit into from

Commits on Sep 14, 2024

  1. test: define default clang args for testing 

    For starters, define the language stardard to use. Use c17 and c++17,
fairly arbitrarily. Tests can override this. We can remove the std
option from template.yaml.

This is a bit more convoluted than should be necessary, for a few
reasons:

- Extension tests use the directive options from directive string. For
  that, the defaults should be specified in conf.py, but hawkmoth_clang
  is not domain specific, and can't be used. See [1]. We need to insert
  the defaults in directive options.

- We want to preserve the original directive options, and present those
  in examples documentation. However, in all testing, we need to amend
  the defaults.

Note that the parser selects the -x language option based on domain and
filename suffix.

[1] #255
    @jnikula
    jnikula committed Sep 14, 2024
    Configuration menu
    Copy the full SHA
    46e767a View commit details
    Browse the repository at this point in the history