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

Chore: Allow 000*.rpy files in renpy/common #300

Merged
merged 1 commit into from
May 7, 2023

Conversation

a2937
Copy link
Contributor

@a2937 a2937 commented May 6, 2023

Chore: Allow 000*.rpy files in renpy/common

Description:

Prevents a 000*.rpy file from the Renpy SDK from being highlighted in red with the error message Filenames must begin with a letter or number, but may not begin with '00' as Ren'Py uses such files for its own purposes.. A file is determined to be part of the sdk if it is under "renpy/common".

Related Issue:

Closes #283

Copy link
Collaborator

@duckdoom4 duckdoom4 left a comment

Choose a reason for hiding this comment

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

Looks good to me. Lgtm 🎉

@duckdoom4 duckdoom4 merged commit 69835d2 into renpy:master May 7, 2023
@a2937 a2937 deleted the renpy-highlight-dia branch May 7, 2023 00:36
@LuqueDaniel LuqueDaniel mentioned this pull request May 7, 2023
2 tasks
@LuqueDaniel LuqueDaniel modified the milestones: 2.2.0, 2.0.17 May 7, 2023
duckdoom4 added a commit that referenced this pull request Jun 2, 2023
* Chore: Allow 000*.rpy files in renpy/common (#300)

* RunRenpyViaplugin (#291)

* Edited extensions method to add way to run renpy

* Added the command to the pallete via json

* Adding debug config to allow renpy to be played from play button

* changed type to cmd

* reformatted with prettier

* This format might work?

* Refactored if statement to make it more readable

* Refactoring of folder names and chaning if statements

* This commit fixed the issues in comment https://github.com/LuqueDaniel/vscode-language-renpy/pull/291#discussion_r1170180867, , refactoring if statement

* reformated through magical means... God I hate switching computers

* Reformated with the correct file, please ignore previous one

* Ignore dist folder (#303)

* Ignore dist folder (#301)

* Chore(Enhancement) - Add a command that cleans up compiled files.

* Update the command title to match the convention.

* Use await/async syntax

* Merge pull request #304 from a2937/old-file-command

Chore(Enhancement) - Add a command that cleans up compiled files.

---------

Co-authored-by: a2937 <a.rcottrill521@gmail.com>
Co-authored-by: seanj29 <88255342+seanj29@users.noreply.github.com>
Co-authored-by: Daniel Luque <danielluque14@gmail.com>
duckdoom4 added a commit that referenced this pull request Jun 2, 2023
* Chore: Allow 000*.rpy files in renpy/common (#300)

* RunRenpyViaplugin (#291)

* Edited extensions method to add way to run renpy

* Added the command to the pallete via json

* Adding debug config to allow renpy to be played from play button

* changed type to cmd

* reformatted with prettier

* This format might work?

* Refactored if statement to make it more readable

* Refactoring of folder names and chaning if statements

* This commit fixed the issues in comment https://github.com/LuqueDaniel/vscode-language-renpy/pull/291#discussion_r1170180867, , refactoring if statement

* reformated through magical means... God I hate switching computers

* Reformated with the correct file, please ignore previous one

* Ignore dist folder (#303)

* Ignore dist folder (#301)

* Chore(Enhancement) - Add a command that cleans up compiled files.

* Update the command title to match the convention.

* Use await/async syntax

* Adds PR ci to master

Fixes #306

* Fixes codeql-analysis.yml format

* Adds setup action

* Fix package lock after merge

---------

Co-authored-by: a2937 <a.rcottrill521@gmail.com>
Co-authored-by: seanj29 <88255342+seanj29@users.noreply.github.com>
Co-authored-by: Daniel Luque <danielluque14@gmail.com>
@LuqueDaniel LuqueDaniel modified the milestones: 2.0.17, 2.2.0 Jun 3, 2023
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.

Red highlight of renpy .rpy files starting with 00
3 participants