Enable globIgnorePaths by default

.github/workflows/ci.yml

@@ -36,10 +36,4 @@ jobs:
       run: |
         cargo fmt -- --check
         cargo clippy
-      if: matrix.rust_version == 'stable'
-
-    - name: Build (All Features)
-      run: cargo build --locked --verbose --all-features
-
-    - name: Run tests (All Features)
-      run: cargo test --locked --verbose --all-features
+      if: matrix.rust_version == 'stable'

CHANGELOG.md

@@ -5,6 +5,8 @@
 * Fixed incorrect string escaping when producing Lua code from JSON files. ([#314](https://github.com/rojo-rbx/rojo/issues/314))
 * Fixed performance issues introduced in Rojo 6.0.0-rc.1. ([#317](https://github.com/rojo-rbx/rojo/issues/317))
 * Updated default place template to take advantage of [#210](https://github.com/rojo-rbx/rojo/pull/210).
+* Enabled glob ignore patterns by default and removed the `unstable_glob_ignore` feature.
+    * `globIgnorePaths` can be set on a project to a list of globs to ignore.
 
 ## [6.0.0 Release Candidate 1](https://github.com/rojo-rbx/rojo/releases/tag/v6.0.0-rc.1) (March 29, 2020)
 This release jumped from 0.6.0 to 6.0.0. Rojo has been in use in production for many users for quite a long times, and so 6.0 is a more accurate reflection of Rojo's version than a pre-1.0 version.

Cargo.toml

@@ -23,9 +23,6 @@ panic = "abort"
 [features]
 default = []
 
-# Turn on support for specifying glob ignore path rules in the project format.
-unstable_glob_ignore_paths = []
-
 # Enable this feature to live-reload assets from the web UI.
 dev_live_assets = []
 

src/project.rs

@@ -61,7 +61,6 @@ pub struct Project {
     /// A list of globs, relative to the folder the project file is in, that
     /// match files that should be excluded if Rojo encounters them.
     #[serde(default, skip_serializing_if = "Vec::is_empty")]
-    #[cfg_attr(not(feature = "unstable_glob_ignore_paths"), serde(skip))]
     pub glob_ignore_paths: Vec<Glob>,
 
     /// The path to the file that this project came from. Relative paths in the

tests/tests/build.rs

@@ -28,6 +28,9 @@ gen_build_tests! {
     csv_in_folder,
     deep_nesting,
     gitkeep,
+    ignore_glob_inner,
+    ignore_glob_nested,
+    ignore_glob_spec,
     infer_service_name,
     infer_starter_player,
     init_meta_class_name,
@@ -48,13 +51,6 @@ gen_build_tests! {
     txt_in_folder,
 }
 
-#[cfg(feature = "unstable_glob_ignore_paths")]
-gen_build_tests! {
-    ignore_glob_inner,
-    ignore_glob_nested,
-    ignore_glob_spec,
-}
-
 fn run_build_test(test_name: &str) {
     let working_dir = get_working_dir_path();