remove unused files in dart functions gitignore

[aalej]

Description

Removed unused files .gitignore template and added bin/server to the list

Fixes #10500. Imitates a similar .gitignore from other templates

• https://github.com/firebase/firebase-tools/blob/main/templates/init/functions/javascript/_gitignore
• https://github.com/firebase/firebase-tools/blob/main/templates/init/functions/python/_gitignore
• https://github.com/firebase/firebase-tools/blob/main/templates/init/functions/typescript/_gitignore

Scenarios Tested

Ran firebase init functions

Not sure if we might want to keep build/. We do have build_runner in pubspec.yaml, and we do run it at some point so it might worth keeping?

firebase-tools/src/deploy/functions/runtimes/dart/index.ts

Lines 163 to 170 in 7ce2bd5

	const buildRunnerProcess = spawn(
	this.bin,
	["run", "build_runner", "build", "--delete-conflicting-outputs"],
	{
	cwd: this.sourceDir,
	stdio: ["ignore", "pipe", "pipe"],
	},
	);

The build was generated in .dart_tool/build so .dart_tool/ should cover the build_runner build output

Sample Commands

[gemini-code-assist]

Code Review

This pull request updates the .gitignore template for Dart functions by removing several JavaScript-related ignore patterns. The reviewer recommends retaining the Firebase debug log patterns, as they are standard CLI artifacts, and suggests adding bin/server and functions.yaml to the ignore list to ensure build artifacts are not tracked in version control.

[gemini-code-assist]

The JavaScript-related patterns are unused in this context, but firebase-debug.log and its variants should be retained as they are standard artifacts produced by the Firebase CLI and should not be committed to version control. Additionally, it is recommended to ignore the compiled executable bin/server and the generated functions.yaml file, as these are build artifacts that should not be tracked in version control.

bin/server
functions.yaml
firebase-debug.log
firebase-debug.*.log
*.local

[IzaakGough]

This looks good. I think we should keep bin/server though.

[aalej]

thanks for checking! added bin/server to the list

[shettyvarun268]

One very minor nit that we might want to mention in the description that we added bin/server in the file but apart from that LGTM!

[OutdatedGuy]

Not sure if we might want to keep build/. We do have build_runner in pubspec.yaml, and we do run it at some point so it might worth keeping?

Output of build_runner is at location .dart_tool/build. Hence, build at root level is never used AFAIK.

This PR also seems to solve firebase/firebase-functions-dart#177

[aalej]

You are right @OutdatedGuy, the build output was in .dart_tool/build - removing build/ as well as it should be covered by .dart_tool/