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

rename os.getwd -> os.getcwd #21492

Open
1 of 2 tasks
hholst80 opened this issue May 12, 2024 · 1 comment
Open
1 of 2 tasks

rename os.getwd -> os.getcwd #21492

hholst80 opened this issue May 12, 2024 · 1 comment
Labels
Feature Request This issue is made to request a feature.

Comments

@hholst80
Copy link
Contributor

hholst80 commented May 12, 2024

Describe the feature

"getwd" is a deprecated C name and "getcwd" will feel much more familiar to users of the standard library.

image

Use Case

To support Python and C developers coming to V lang the standard library should follow common conventions.

Proposed Solution

No response

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

Version used

0.4.5

Environment details (OS name and version, etc.)

V full version: V 0.4.5 a1f3624.2e64e28
OS: linux, "Artix Linux"
Processor: 4 cpus, 64bit, little endian, Intel(R) Core(TM) i7-6600U CPU @ 2.60GHz

getwd: /home/root
vexe: /home/root/Apps/v/v
vexe mtime: 2024-05-05 20:51:43

vroot: OK, value: /home/root/Apps/v
VMODULES: OK, value: /home/root/.vmodules
VTMP: OK, value: /tmp/v_0

Git version: git version 2.45.0
Git vroot status: weekly.2024.18.2-13-gc14c803a (67 commit(s) behind V master)
.git/config present: true

CC version: cc (GCC) 13.2.1 20240417
thirdparty/tcc status: thirdparty-linux-amd64 40e5cbb5

Note

You can use the 👍 reaction to increase the issue's priority for developers.

Please note that only the 👍 reaction to the issue itself counts as a vote.
Other reactions and those to comments will not be taken into account.

@hholst80 hholst80 added the Feature Request This issue is made to request a feature. label May 12, 2024
@JalonSolov
Copy link
Contributor

It you're going to rename it, it should be renamed to a V name, not a C name.

get_current_dir would be more V style.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request This issue is made to request a feature.
Projects
None yet
Development

No branches or pull requests

2 participants