-
Notifications
You must be signed in to change notification settings - Fork 4
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
Answers -> Search renaming #116
Conversation
This PR adds description and keywords to package.json. The description is copied from README and keywords provided by product. J=SLAP-2190 TEST=manual
* Customer facing renaming * API extracted * update test site * Update index.ts * Addressed comments * fix provideHeadless * API update for SearchHeadless * Automated update to repo's documentation from github action * update exports * export from deprecated * renamed files * file edits * rm ds store * gitignore fix * minor fixes * Automated update to repo's documentation from github action Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* Customer facing renaming * API extracted * update test site * Update index.ts * Addressed comments * fix provideHeadless * API update for SearchHeadless * Automated update to repo's documentation from github action * answers-core -> search-core * update exports * export from deprecated * renamed files * file edits * rm ds store * gitignore fix * minor fixes * Automated update to THIRD-PARTY-NOTICES from github action's 3rd party notices check * Automated update to repo's documentation from github action * export deprecated answers-core things * fix link * Automated update to repo's documentation from github action * replaced some imports from search-core in index * rename answers-core.ts * update lib and meta.ts * Automated update to repo's documentation from github action Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Could we have a PR to update variable names and test fixture names? If that's going to be a separate thing lgtm! |
Yeah I just confirmed with Tom yesterday, internal stuff is going to be another phase of the renaming :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should be merged into develop
instead of main
Is the npm package going to be from |
we would merge into |
merge with develop .
aah got it ty! |
Pull Request Test Coverage Report for Build 2665475341
💛 - Coveralls |
J=SLAP-2204
TEST=auto