Skip to content
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

Feature/sc 253631/add databricks code structure in analytics #317

Merged

Conversation

aarroyosal
Copy link
Contributor

@aarroyosal aarroyosal commented Sep 14, 2022

Description

Shortcut

This PR includes the changes needed in order to include the databricks structure in AT advanced
It also adds headers to the functions that didn't have and fixes a few doc files

Changed

  • change makefiles and sbt file to prepare the at advance

Added

  • add headers to functions that didn't have

Fixed

  • formatters doc
  • Some parser tests that had old user function names

Acceptance

cd clouds/databricks
make publish-local-core
The previous command should install the core scala libraries in $HOME/.ivi local repository
image

make ci-release-core
This command should publish a snapshot version of core in the sonatype releases repository
https://s01.oss.sonatype.org/content/repositories/snapshots/com/carto/analyticstoolbox/core_2.12/
image

make check-header | make create-header
This commands check if all the classes have header with license and create them in case they don't

@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #253631: Add Databricks code structure in analytics-toolbox.

@aarroyosal aarroyosal requested a review from Jesus89 September 16, 2022 08:43
@aarroyosal aarroyosal requested a review from a team as a code owner September 16, 2022 08:48
@aarroyosal aarroyosal merged commit 7ab5d6f into master Sep 19, 2022
@aarroyosal aarroyosal deleted the feature/sc-253631/add-databricks-code-structure-in-analytics branch September 19, 2022 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants