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

Move folder: src/core/utils/src/utils/ #508

Merged
merged 9 commits into from
Oct 23, 2023
Merged

Move folder: src/core/utils/src/utils/ #508

merged 9 commits into from
Oct 23, 2023

Conversation

Raruto
Copy link
Collaborator

@Raruto Raruto commented Oct 19, 2023

Preparatory for: #505

List of changes

  • move whole files in folder: src/core/utils/*.jssrc/utils/*.js
  • extract some functions into single files: src/core/utils/utils.jssrc/utils/*.js
  • remove unusued module: utils/ol.controls'

Before

const utils    = require('core/utils/utils');
const geoutils = require('core/utils/geo');
const parsers  = require('core/utils/parsers');

After

const utils    = require('utils');
const geoutils = require('utils/geo');
const parsers  = require('utils/parsers');

@Raruto Raruto added the refactoring Anything which could result in a API change label Oct 19, 2023
@Raruto Raruto added this to the v3.9 milestone Oct 19, 2023
@volterra79 volterra79 assigned volterra79 and Raruto and unassigned volterra79 Oct 23, 2023
@Raruto Raruto merged commit a06d14d into dev Oct 23, 2023
@Raruto Raruto deleted the move-utils-folder branch October 23, 2023 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Anything which could result in a API change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants