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

Snakemake files have Python syntax #2745

Closed
wants to merge 1 commit into from
Closed

Snakemake files have Python syntax #2745

wants to merge 1 commit into from

Conversation

slowkow
Copy link

@slowkow slowkow commented Nov 24, 2015

In #1834 the filename Snakefile was added to the list of Python filenames.

Here, I propose to add 3 file extensions for Snakemake. All 3 of these file extensions are unique to Python.

The author of Snakemake lists the following file extensions for Snakefiles:

  • .rules
  • .snakefile
  • .snake

and the following filenames:

  • Snakefile

Aside: In the future, when Snakemake is more mature, it may be sensible to provide a language grammar based on Python and customized for Snakemake. Currently, I favor highlighting Snakemake files like Python files.

In #1834 the filename `Snakefile` was added to the list of Python filenames.

Here, I propose to add 3 file extensions for [Snakemake][1]. All 3 of these file extensions are unique to Python.

The author of Snakemake lists the following [file extensions for Snakefiles][2]:

- .rules
- .snakefile
- .snake

and the following filenames:

- Snakefile

Aside: In the future, when Snakemake is more mature, it may be sensible to provide a language grammar based on Python and customized for Snakemake. Currently, I favor highlighting Snakemake files like Python files.

[1]: https://bitbucket.org/snakemake/snakemake/wiki/Home
[2]: https://bitbucket.org/snakemake/snakemake/wiki/FAQ#markdown-header-how-do-i-enable-syntax-highlighting-in-vim-for-snakefiles
@arfon
Copy link
Contributor

arfon commented Nov 25, 2015

Hi @slowkow thanks for opening this Pull Request. Firstly, a little housekeeping: the tests are current failing as the extensions need to be sorted alphabetically, if you could fix that up that would be great.

Secondly, would you mind reviewing our contributing guidelines? Specifically it would be great to see links to search queries showing in-the-wild usage of these extensions.

@arfon
Copy link
Contributor

arfon commented Dec 3, 2015

🎏 flagging this as stale 🎏

@arfon arfon closed this Dec 15, 2015
@nh13
Copy link
Contributor

nh13 commented Dec 17, 2017

This PR has been picked up here: #3953

@github-linguist github-linguist locked as resolved and limited conversation to collaborators Jun 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants