also provide TypeScript declarations in types/index.d.ts. In addition, we have a TypeScript test suite in test-dts/index.test-d.ts. When reviewing pull requests, please verify that any changes to the JavaScript
No other project uses it yet
3.5k41d agoReviewMountCreate a free account to mount
info here.
## Working Effectively
**Prerequisites:**
- Go >= 1.27 (currently uses Go 1.27.1)
- Helm v3 (tested with v3.17.4 and v3.18.6)
**Build Process:**
- Build the plugin: `make build` - includes linting and compiles
3.5k10d agoReviewMountCreate a free account to mount
other applications
- Support customizable hotkeys
## Project Structure
- Use separate assembly projects for distinct functionality
- Tests should be in corresponding test projects
- UI components should be separated from business logic
## Important
3.5k12d agoReviewMountCreate a free account to mount
WMFDataMocks
WMFData produces another library called WMFDataMocks, which holds our mocked classes for unit testing. We eventually also want to use these mocked classes in the app itself, which will
3.5k39d agoReviewMountCreate a free account to mount
support lifecycles for various products. The site is built and deployed to Netlify.
## Build, Test, and Lint Commands
### Development Server
```bash
# Install dependencies (first time only)
bundle install
# Run site
3.4k8mo agoReviewMountCreate a free account to mount
Swagger/OpenAPI generated docs for manager REST API
version/ # Version variables injected at build time
test/
e2e/ # End-to-end tests using Ginkgo (require Docker)
testdata/ # Shared test fixtures
deploy/
docker
3.4k3y agoReviewMountCreate a free account to mount
code, answering questions, or reviewing changes.
Whenever you want to build the packages to test if they work you should run `pnpm run build` from the root of the repository
Steady7 mentions in 2 weeks
No other project uses it yet
3.4k25d agoReviewMountCreate a free account to mount
/DCO, and coverage
percentages: automation already enforces or reports these.
- Generated, vendored, or test fixture data files (including `dat/`).
- Readability or maintainability opinions without a concrete failure mode.
- Verbose
3.4k7mo agoReviewMountCreate a free account to mount
loaded in the session,
> start a new PowerShell session before re-importing the module.
## Testing
Use `test.ps1`, which runs a build and imports required helpers before starting:
```powershell
.\test.ps1 -File
3.4k5y agoReviewMountCreate a free account to mount
unexpected information that does not match the info here.**
## Working Effectively
### Bootstrap, Build, and Test the Repository
1. **Environment Setup (Required for all development):**
```bash
# Copy environment file
3.4k21mo agoReviewMountCreate a free account to mount
GOOD: `NameIdentifier id = ...` (add `import org.apache.gravitino.NameIdentifier;`)
- **Coding Style**: Strict adherence to Google Java Style.
- **Testing**:
- EVERY change must have a corresponding test.
- Do NOT modify existing tests unless the logic
3.4k5mo agoReviewMountCreate a free account to mount
review
- **Generated files are never hand-edited:** `Source/**/CompatibilitySuppressions.xml` (ApiCompat baselines) and `linq2db.baselines` test baselines are tool output. A hand-written change to them is a finding.
- **New or changed
Steady14 mentions in 2 weeks
No other project uses it yet
3.3k15mo agoReviewMountCreate a free account to mount
monorepo containing the core framework, CLI tools, modules, integrations, examples, and comprehensive tests.
## Pull Request Automation
When working with pull requests in this repository, **ALWAYS**:
1. **Use the PR description
No other project uses it yet
3.3k22mo agoReviewMountCreate a free account to mount
Project general coding standards
## Testing
- Write unit tests for all new functionality
- Ensure existing tests pass before submitting changes
- Use mocks and stubs where appropriate to isolate units under test
3.3k7mo agoReviewMountCreate a free account to mount
error.
## What to review
Correctness and bugs, blocking IO in async code, security, performance, test coverage, docs the change makes wrong, and consistency with the surrounding code. `AGENTS.md` holds
Steady6 mentions in 2 weeks
No other project uses it yet
3.3k4mo agoReviewMountCreate a free account to mount