Skip to content

Checking CI #6202

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 11 commits into from
Closed

Checking CI #6202

wants to merge 11 commits into from

Conversation

naterush
Copy link

Just checking CI.

mentatbot bot and others added 11 commits March 18, 2025 21:31
@mentatbot
ci: add setup and precommit scripts for automated development
0afebe3 
This PR adds two shell scripts to facilitate automated development:

1. `.mentat/setup.sh`: Installs all necessary dependencies a developer would need to run tests and the project.
   - Installs Node.js dependencies
   - Sets up development environment
   - Checks for Node.js installation

2. `.mentat/precommit.sh`: Runs checks before committing code to ensure code quality.
   - Runs ESLint with auto-fix on staged JavaScript files
   - Lints Markdown files using Remark
   - Checks commit message format
   - Runs targeted tests for affected packages

These scripts focus on the most critical development tasks while avoiding redundancy with GitHub CI. The setup script ensures developers can quickly get started, while the precommit script enforces code quality standards without running the full test suite (which is handled by GitHub CI).

The .gitignore file has also been updated to include the .mentat directory.

Both scripts have a 4-minute timeout limit as specified in the requirements.

Mentat precommits passed. Log: https://mentat.ai/log/6145f1a7-de6b-4a99-a9dd-ae31ebb0b7e9
@naterush
Merge pull request #1 from human-uplift/mentat-scripts-1
d6e64de 
ci: add setup and precommit scripts for automated development
@naterush
Update lint_pr_title.yml
47bcd74 
Signed-off-by: Nate Rush <[email protected]>
@naterush
Update labeler.yml
d87a3da 
Signed-off-by: Nate Rush <[email protected]>
@naterush
Update first_time_greeting.yml
3fdf275 
Signed-off-by: Nate Rush <[email protected]>
@naterush
Update check_contributing_guidelines_acceptance.yml
c8fa4e7 
Signed-off-by: Nate Rush <[email protected]>
@naterush
cleanup
8bd9e47
@naterush
feat: add new test
c718143 
---
type: pre_commit_static_analysis_report
description: Results of running static analysis checks when committing changes.
report:
  - task: lint_filenames
    status: passed
  - task: lint_editorconfig
    status: passed
  - task: lint_markdown
    status: na
  - task: lint_package_json
    status: na
  - task: lint_repl_help
    status: na
  - task: lint_javascript_src
    status: na
  - task: lint_javascript_cli
    status: na
  - task: lint_javascript_examples
    status: na
  - task: lint_javascript_tests
    status: passed
  - task: lint_javascript_benchmarks
    status: na
  - task: lint_python
    status: na
  - task: lint_r
    status: na
  - task: lint_c_src
    status: na
  - task: lint_c_examples
    status: na
  - task: lint_c_benchmarks
    status: na
  - task: lint_c_tests_fixtures
    status: na
  - task: lint_shell
    status: missing_dependencies
  - task: lint_typescript_declarations
    status: na
  - task: lint_typescript_tests
    status: na
  - task: lint_license_headers
    status: passed
---
@naterush
fix: remove unnecessary workflows
4935aa8 
---
type: pre_commit_static_analysis_report
description: Results of running static analysis checks when committing changes.
report:
  - task: lint_filenames
    status: passed
  - task: lint_editorconfig
    status: passed
  - task: lint_markdown
    status: na
  - task: lint_package_json
    status: na
  - task: lint_repl_help
    status: na
  - task: lint_javascript_src
    status: na
  - task: lint_javascript_cli
    status: na
  - task: lint_javascript_examples
    status: na
  - task: lint_javascript_tests
    status: na
  - task: lint_javascript_benchmarks
    status: na
  - task: lint_python
    status: na
  - task: lint_r
    status: na
  - task: lint_c_src
    status: na
  - task: lint_c_examples
    status: na
  - task: lint_c_benchmarks
    status: na
  - task: lint_c_tests_fixtures
    status: na
  - task: lint_shell
    status: na
  - task: lint_typescript_declarations
    status: na
  - task: lint_typescript_tests
    status: na
  - task: lint_license_headers
    status: passed
---
@naterush
Merge pull request #4 from human-uplift/test-pr
ea02b3e 
cleanup
@naterush
fix: remove unused test
3deeb21 
---
type: pre_commit_static_analysis_report
description: Results of running static analysis checks when committing changes.
report:
  - task: lint_filenames
    status: passed
  - task: lint_editorconfig
    status: passed
  - task: lint_markdown
    status: na
  - task: lint_package_json
    status: na
  - task: lint_repl_help
    status: na
  - task: lint_javascript_src
    status: na
  - task: lint_javascript_cli
    status: na
  - task: lint_javascript_examples
    status: na
  - task: lint_javascript_tests
    status: passed
  - task: lint_javascript_benchmarks
    status: na
  - task: lint_python
    status: na
  - task: lint_r
    status: na
  - task: lint_c_src
    status: na
  - task: lint_c_examples
    status: na
  - task: lint_c_benchmarks
    status: na
  - task: lint_c_tests_fixtures
    status: na
  - task: lint_shell
    status: na
  - task: lint_typescript_declarations
    status: na
  - task: lint_typescript_tests
    status: na
  - task: lint_license_headers
    status: passed
---
@naterush naterush closed this Mar 19, 2025
@stdlib-bot
Copy link
Contributor

Hello! Thank you for your contribution to stdlib.

We noticed that the contributing guidelines acknowledgment is missing from your pull request. Here's what you need to do:

  1. Please read our contributing guidelines.

  2. Update your pull request description to include this checked box:

    - [x] Read, understood, and followed the [contributing guidelines](https://github.com/stdlib-js/stdlib/blob/develop/CONTRIBUTING.md)

This acknowledgment confirms that you've read the guidelines, which include:

  • The developer's certificate of origin
  • Your agreement to license your contributions under the project's terms

We can't review or accept contributions without this acknowledgment.

Thank you for your understanding and cooperation. We look forward to reviewing your contribution!

@stdlib-bot stdlib-bot added Needs Review A pull request which needs code review. and removed Needs Review A pull request which needs code review. labels Mar 19, 2025
@naterush naterush deleted the cleanup-1 branch March 19, 2025 18:03
stdlib-bot
stdlib-bot reviewed Mar 19, 2025
View reviewed changes
Copy link
Contributor

@stdlib-bot stdlib-bot left a comment

Choose a reason for hiding this comment

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

👋 Hi there! 👋

And thank you for opening your first pull request! We will review it shortly. 🏃 💨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stdlib-bot stdlib-bot stdlib-bot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@naterush @stdlib-bot