summaryrefslogtreecommitdiffstats
path: root/.github
Commit message (Collapse)AuthorAgeFilesLines
* Fix various typos with codespellRémi Verschelde2023-06-161-2/+2
| | | | And ignore some false positives introduced by recent versions of codespell.
* Link to Code style guidelines in the pull request templateHugo Locurcio2023-06-151-0/+3
| | | | | This is a common pain point for new contributors, as setting up pre-commit hooks will save a lot of time spent on CI.
* Explain in GitHub issue template how to copy exact Godot versionkleonc2023-05-301-2/+2
|
* Make system information for issue reports single-lineMJacred2023-05-291-10/+3
| | | | | | | Additionally: * for custom builds, add commit hash * added a tooltip to help mitigate potential/understandable confusion whether the single-line output might be a bug * prettify driver name
* Merge pull request #65902 from MJacred/editor/copysysteminfoYuri Sizov2023-05-271-4/+12
|\ | | | | Add an editor option to copy system info to clipboard
| * Copy system info to clipboard + Update bug_report.ymlMJacred2023-05-271-4/+12
| | | | | | | | | | | | | | | | plus minor static-related fixes * linuxbsd: get_systemd_os_release_info_value() -> static breaks usage if used multiple times * windows/linuxbsd: get_video_adapter_driver_info() writes info into static * linuxbsd: get_distribution_name() + get_version() -> write bsd fallback into static variable * windows/uwp/android: remove unnecessary use of static
* | Allow validating the extension api against a reference version and a list of ↵RedworkDE2023-05-191-0/+6
|/ | | | known changes.
* CI: Use GODOT_BASE_BRANCH for the godot-cpp checkoutRémi Verschelde2023-05-186-6/+7
|
* Merge pull request #76887 from RedworkDE/ci-fix-windows-unit-testsRémi Verschelde2023-05-151-1/+1
|\ | | | | CI: Fix running the unit tests on windows
| * CI: Fix running the unit tests on windowsRedworkDE2023-05-101-1/+1
| |
* | CI: Skip codespell check if `CHANGED_FILES` is emptyRémi Verschelde2023-05-101-2/+2
|/ | | | | This can happen when only thirdparty files are changed, since we grep them out. Re-add `bin` and `thirdparty` to the skip list for good measure.
* CI: Use gh-cli for changed files, and workaround codespell skip list bugRémi Verschelde2023-05-081-5/+6
| | | | | | | | | For PRs, this should give a more accurate list, as the previous method would diff to the tip of the `master` branch, which could include new commits (and thus changed files) not present in the PR branch. codespell's `--skip` option doesn't work at all with folders when used together with an explicit list of paths to work with, so let's not use it.
* CI: Make codespell checks blocking, but only check changed filesRémi Verschelde2023-05-081-2/+5
|
* Streamline and centralize bug reporting guidesJason Yundt2023-05-031-8/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | Before this change, there was three different bug reporting guides: - [“Filing an issue on GitHub”][1] - [“Reporting bugs”][2] - [The issue template][3] This commit: 1. makes sure that [the issue template][3] contains all of the same information that [“Filing an issue on GitHub”][1] and [“Reporting bugs”][2] did and 2. makes [“Reporting bugs”][2] simply tell users to fill out the template. The goal of this change is to make reporting bugs easier. This change accomplishes that goal by presenting bug reporters with all of the information they need to know on the bug reporting page itself. This commit partially implements this proposal: <https://github.com/godotengine/godot-proposals/discussions/4083> [1]: https://docs.godotengine.org/en/stable/community/contributing/ways_to_contribute.html#filing-an-issue-on-github [2]: ./CONTRIBUTING.md#reporting-bugs [3]: ./.github/ISSUE_TEMPLATE/bug_report.yml
* Create an "Editor-only" section in the online class referenceYuri Sizov2023-04-241-1/+1
| | | | | | | | | This helps to find such classes without digging through the rest of the class reference. Editor-only classes can still be found under your normal "Node" and "Resource" types. This also fixes a typo and a missed case from the recent platform docs PR.
* CI: Fix checks fallback logic when changed files listing is missingRémi Verschelde2023-04-201-4/+4
| | | | The file would be one byte (newline) so the check with '-s' failed.
* CI: Fixup failure to get changed files on mergeRémi Verschelde2023-04-201-2/+2
| | | | | | | | | | The checkout might be too shallow so the before commit isn't available. The logic was already written to take this into account (it then generates an empty 'changed.txt' which falls back to testing everything), but the error code would still force terminate the job. Hopefully we can find a way to make the logic work for merge events too in the future, but for now this is a quick fix.
* Merge pull request #76263 from akien-mga/ci-gotta-go-fast-with-styleRémi Verschelde2023-04-202-21/+50
|\ | | | | | | CI: Speed up static checks by checking only changed files
| * CI: Speed up static checks by checking only changed filesRémi Verschelde2023-04-202-21/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - file_format, header_guards and clang-format benefit from this short list. - dotnet-format, Python and JS checks don't, but they're only relevant for PRs changing a specific set of files, so we skip them when those files aren't modified. The logic to get changed files only works reliably for: - Pull request events - Non-force pushed push events So when force pushing a branch in your fork, or creating a new branch, it will still scan all files as fallback. Upgraded CI runner to Ubuntu 22.04 so we get clang-format 14 out of the box, so we don't need to install a custom version (saves ~15 s). We also cache the APT dependencies to speed up the build and avoid flaky Ubuntu/Microsoft repos.
* | [Export docs] Move docs to platform folders.bruvzg2023-04-201-2/+2
|/
* Merge pull request #75700 from akien-mga/ci-visible-whitespaceRémi Verschelde2023-04-051-1/+2
|\ | | | | | | CI: Visibly print trailing whitespace when static checks fail
| * CI: Visibly print trailing whitespace when static checks failRémi Verschelde2023-04-051-1/+2
| | | | | | | | | | | | | | | | | | GitHub Actions seems to be hiding colored whitespace, and after lots of attempts I couldn't find a way to work it around. So instead I'm using a perl expression to replace trailing spaces with `·` and tabs with `<TAB>` in the ANSI colored diff output. This ensure that they're visible, and they are properly colored as expected too.
* | [ci/windows] remove obsolete 'tools' in the nameumarcor2023-04-051-1/+1
|/
* ci: wait for static check results before starting buildsyedpodtrzitko2023-04-048-7/+55
|
* [ci/web] update mymindstorm/setup-emsdk to v12Unai Martinez-Corral2023-03-261-1/+1
|
* CI: Pin SCons to 4.4.0, the new 4.5.0 is brokenRémi Verschelde2023-03-061-1/+1
|
* CI: Disable debug_symbols for Linux + all sanitizers buildRémi Verschelde2023-02-181-1/+2
| | | | We're running out of disk space so builds are failing...
* CI: Remove flaky packages.microsoft.com Ubuntu repositoryRémi Verschelde2023-02-112-2/+2
| | | | We don't need it.
* CI: Remove custom Linux deps and SwiftShaderRémi Verschelde2023-02-063-35/+9
| | | | | | | | The default environment already includes everything we need to build all our configurations. Remove custom SwiftShader setup as lavapipe should now be good enough, but we need to install the latest one.
* CI: Add official codespell action with PR annotationsRémi Verschelde2023-02-011-0/+8
|
* [CI] Remove packages that are no longer necessary.bruvzg2023-01-251-4/+2
|
* Cleanup and unify keyboard input.bruvzg2023-01-231-1/+1
| | | | | | | | | | - Unify keycode values (secondary label printed on a key), remove unused hardcoded Latin-1 codes. - Unify IME behaviour, add inline composition string display on Windows and X11. - Add key_label (localized label printed on a key) value to the key events, and allow mapping actions to the unshifted Unicode events. - Add support for physical keyboard (Bluetooth or Sidecar) handling on iOS. - Add support for media key handling on macOS. Co-authored-by: Raul Santos <raulsntos@gmail.com>
* Fix file formatting script dependencies and cleanupAaron Franke2023-01-121-1/+1
|
* CI: Use clang-format 15Rémi Verschelde2023-01-101-3/+3
| | | | No change compared to version 13 in our codebase.
* CI: Fix dumping GDExtension interface and API for godot-cppRémi Verschelde2022-12-141-12/+5
| | | | | | | | | Follow-up to https://github.com/godotengine/godot-cpp/pull/960. Fix exit code for --dump-extension-api and --dump-gdextension-interface. Removed the planned API validation step as we still didn't implement anything, and maintaining a stub isn't useful.
* Merge pull request #69718 from groud/finally_rename_gdnative_to_gdextensionRémi Verschelde2022-12-122-3/+2
|\ | | | | | | Rename all gdnative occurences to gdextension
| * Rename all gdnative occurences to gdextensionGilles Roudière2022-12-122-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | Non-exhaustive list of case-sensitive renames: GDExtension -> GDNative GDNATIVE -> GDEXTENSION gdextension -> gdnative ExtensionExtension ->Extension (for where there was GDNativeExtension) EXTENSION_EXTENSION ->EXTENSION (for where there was GDNATIVE_EXTENSION) gdnlib -> gdextension gdn_interface -> gde_interface gdni -> gde_interface
* | Rename `float=64` SCons option to `precision=double`Hugo Locurcio2022-12-101-2/+2
|/ | | | | This avoids confusion with the old `bits=64` option and building for 64-bit CPUs in general.
* Emphasize the importance of uploading a minimal reproduction projectHugo Locurcio2022-11-171-2/+5
| | | | | | The field is now required, but "N/A" can be manually entered if the reproduction steps are trivial and don't require any project files to be followed.
* Merge pull request #68543 from DarkKilauea/msvc-problem-matcherRémi Verschelde2022-11-111-1/+3
|\
| * Add MSVC problem matcher to CIJosh Jones2022-11-111-1/+3
| |
* | Add GCC problem matcher to CIRaul Santos2022-11-111-0/+3
| |
* | Mention .godot folder in issue templatekobewi2022-11-111-1/+1
|/
* Add rendering team as GLES3 CODEOWNERclayjohn2022-11-021-0/+1
|
* Fix MSVC warnings, rename shadowed variables, fix uninitialized values, ↵bruvzg2022-10-071-1/+1
| | | | change warnings=all to use /W4.
* Various enhancements to Visual Studio solution generation.TechnoPorg2022-10-041-1/+1
| | | | | This adds support for building solutions with dev_mode and/or float=64 enabled. Additionally, it adds solution generation to the Windows CI to catch future regressions.
* CI: Update target for godot-cpp after ↵Rémi Verschelde2022-10-041-1/+1
| | | | https://github.com/godotengine/godot-cpp/pull/867
* Fix CODEOWNERS and fix missing quote in "Building for platform" messageAaron Franke2022-10-011-2/+2
|
* ci: add Python static analysis check via mypyJiri Suchan2022-09-301-1/+5
|
* SCons: Unify tools/target build type configurationRémi Verschelde2022-09-267-61/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements https://github.com/godotengine/godot-proposals/issues/3371. New `target` presets ==================== The `tools` option is removed and `target` changes to use three new presets, which match the builds users are familiar with. These targets control the default optimization level and enable editor-specific and debugging code: - `editor`: Replaces `tools=yes target=release_debug`. * Defines: `TOOLS_ENABLED`, `DEBUG_ENABLED`, `-O2`/`/O2` - `template_debug`: Replaces `tools=no target=release_debug`. * Defines: `DEBUG_ENABLED`, `-O2`/`/O2` - `template_release`: Replaces `tools=no target=release`. * Defines: `-O3`/`/O2` New `dev_build` option ====================== The previous `target=debug` is now replaced by a separate `dev_build=yes` option, which can be used in combination with either of the three targets, and changes the following: - `dev_build`: Defines `DEV_ENABLED`, disables optimization (`-O0`/`/0d`), enables generating debug symbols, does not define `NDEBUG` so `assert()` works in thirdparty libraries, adds a `.dev` suffix to the binary name. Note: Unlike previously, `dev_build` defaults to off so that users who compile Godot from source get an optimized and small build by default. Engine contributors should now set `dev_build=yes` in their build scripts or IDE configuration manually. Changed binary names ==================== The name of generated binaries and object files are changed too, to follow this format: `godot.<platform>.<target>[.dev][.double].<arch>[.<extra_suffix>][.<ext>]` For example: - `godot.linuxbsd.editor.dev.arm64` - `godot.windows.template_release.double.x86_64.mono.exe` Be sure to update your links/scripts/IDE config accordingly. More flexible `optimize` and `debug_symbols` options ==================================================== The optimization level and whether to generate debug symbols can be further specified with the `optimize` and `debug_symbols` options. So the default values listed above for the various `target` and `dev_build` combinations are indicative and can be replaced when compiling, e.g.: `scons p=linuxbsd target=template_debug dev_build=yes optimize=debug` will make a "debug" export template with dev-only code enabled, `-Og` optimization level for GCC/Clang, and debug symbols. Perfect for debugging complex crashes at runtime in an exported project.