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

Removal of unnecessary test files #80

Merged
merged 1 commit into from Oct 8, 2020
Merged

Removal of unnecessary test files #80

merged 1 commit into from Oct 8, 2020

Conversation

megatank58
Copy link
Contributor

@megatank58 megatank58 commented Oct 8, 2020

Removal of unnecessary Tests folder, InputTestScene and the content in the Scene. I have tested the game and it doesn't breaks or effects the game in any possible way.

Forum thread: Lets stick to contribution guidelines
Removal of unnecessary test files,
As per the community guidelines we are not supposed to make such testing scenes.
This has been fixed by deleting this, i have tested this and it doesn't introduce any errors for any major system of game.

Removal of unnecessary Tests folder, Input Scene and the content in the Scene. I have tested the game and it doesn't breaks or effects the game in any possible way.
@ciro-unity
Copy link
Contributor

Good point, thanks!

@ciro-unity ciro-unity merged commit 2230724 into UnityTechnologies:master Oct 8, 2020
STARasGAMES added a commit to STARasGAMES/open-project-1 that referenced this pull request Oct 15, 2020
* My take on a simple Player Spawn System

* Renamed SetRequestSpawnIndex to SetSpawnIndex

* Fixed steep surfaces bug

* Update UOP1_Project/Assets/Scripts/Characters/Character.cs

I deleted unnecessary spacing by following atb-brown suggestion.

Co-authored-by: Austin Brown <atb.austin@gmail.com>

* Deleted unnecessary spacing

* Made adjustments based on HyagoOliveiran's suggestions

Tried to add more variables to increase legibility.
Added UpdateSlide function.
Deleted some empty lines.

* Update Character.cs

* Update Character.cs

* Update Character.cs

* Update Character.cs

* Update Character.cs

* Update Character.cs

* InputReader made into ScriptableObject asset

* Replace magic numbers with variables - issue: 40

* Fixed bouncing issue when player hit under floating platform

* Change var name to more acurately show purpose

* Remove accidentaly added blank lines

* Spawn system in place

* SpawnSystem active in the scene

Removed the player prefab from the scene

* Changed the new variable into a constant

* Implemented automatic linter

* Removed bot account from auto commit

* [Bot] Automated dotnet-format update

* Create pull_request_template.md

* Moved .editorConfig to avoid duplicated version in actual project root (UnityTechnologies#77)

* Removal of unnecessary test files (UnityTechnologies#80)

Removal of unnecessary Tests folder, Input Scene and the content in the Scene. I have tested the game and it doesn't breaks or effects the game in any possible way.

* Removed unused imports (UnityTechnologies#75)

* Project settings

Small tweaks, project naming, etc.

* Update README.md (UnityTechnologies#82)

Added a permanent and non-capped invite link to the #open-projects Discord channel on the Official Unity Discord server.

Co-authored-by: Wodopo <pedrocoelhotv@gmail.com>
Co-authored-by: mwert09 <mwert09@gmail.com>
Co-authored-by: Mert <46690577+mwert09@users.noreply.github.com>
Co-authored-by: Austin Brown <atb.austin@gmail.com>
Co-authored-by: Andrzej Łukasik <andrew.r.lukasik@gmail.com>
Co-authored-by: Andrew Real <andrew@andrewreal.com>
Co-authored-by: Ciro Continisio <ciro@unity3d.com>
Co-authored-by: Miley Hollenberg <mileyhollenberg@gmail.com>
Co-authored-by: Miley Hollenberg <miley.hollenberg@unity3d.com>
Co-authored-by: Megatank58 <51410502+MEGATANK58@users.noreply.github.com>
Co-authored-by: Wikum Chamith <wikum@linuxdeveloper.space>
Co-authored-by: Dave Rodriguez <drod7425@gmail.com>
@ciro-unity ciro-unity added the enhancement New feature or request label Oct 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants