6.7.0 release#1010
Merged
juliannguyen4 merged 235 commits intostagefrom Apr 28, 2026
Merged
Conversation
…need to set this attribute and as_query.ops will be set behind the scenes. This will enable bin projection if they call foreach/results()
…d still figuring out how the test generators work
…ys want to bump the version in a feature branch i.e in dev-workflow.yml..
…t a registry name to pull images from for better flexibility
…e one package.json for all package and dev dependencies. I believe this should work since if there's no package.json in ts-test, it will move up to the repo's root dir which does have the dev dependencies.
…les the typescript tests in ts-test/, and we shouldn't compile any typescript outside ts-test/ in order to run the tests
…lient on a specific platform. This is to save energy
…odules folder. So we don't need to use npx here.
…sense, assuming aerospike module is not installed in the repo's node_modules/ folder.
…aths, use tsc-alias to replace all of whem with relative paths in the compiled js files. Otherwise, node.js will just look for aerospike in node_modules which only contains dependencies.
…n on pushes to dev branch
…ule test instead of the actual client
…sed, and they were exclusively in ts-test/package.json so add them to the root-level package json
… commit and this package weren't in the top level package json and I could still compile the ts tests...?
…pike module test instead of the actual client" This reverts commit 4295c97.
…spike module test instead of the actual client". The CI/CD results confirm that the test code is importing the aerospike.ts test file instead of the node.js client. This reverts commit 843177b.
…ondering if main is set wrong here?
…em like redundant paths so just remove
…find the .js files that map to the .ts files
… replace all the aerospike imports with relative imports
…s sense. That way tsc can validate the tests are using the node.js client properly as well as actually import the client at runtime
…ocation and not where it was run from
…se flags will be reset for future test cases
…king the cluster returned from that
…ntain permissions' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…d to specify the exact tag from artifact.aerospike.io
justinlee-aerospike
previously approved these changes
Apr 27, 2026
Contributor
justinlee-aerospike
left a comment
justinlee-aerospike
left a comment
There was a problem hiding this comment.
changes are okay on first review. Let's have a debrief on all of the tech debt fixes that were needed for this after release.
justinlee-aerospike
approved these changes
Apr 28, 2026
Contributor
justinlee-aerospike
left a comment
justinlee-aerospike
left a comment
There was a problem hiding this comment.
change looks good
juliannguyen4
added a commit
that referenced
this pull request
Apr 28, 2026
Add support for server 8.1.2 features: - Creating set indexes via the client API - Bin projection - Path expressions quality of life enhancements Changes in addition to ones made in 8.1.2 feature PR's - CI/CD: Allow selecting the bump type when bumping the dev number after a release version Add back script to delete .node files that aren't relevant to the user's platform. This is a temporary fix until we start using an S3 bucket - While working on this release, I started noticing the tests were running out of order in mocha. This caused the admin test cases to fail because they depended on the ordering of the test cases. So for admin.ts, I changed the test cases so that they can run independently of each other - admin.ts: add skipAll helper to avoid conflicting with beforeEach fixtures
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes in addition to ones made in 8.1.2 feature PR's
skipAllhelper to avoid conflicting with beforeEach fixturesTODO
Script doesn't seem to trigger on test-eeMetrics client version test is failing; suspect that the wrong JFrog build is being pulleddevhere and server 8.1.1. This means the memory error wasn't introduced in v6.7.0, but we need to add this error to a suppressions file for nowKnown Issues
mapKeysInis returning a server errorOut of scope